Closed Bug 1270029 Opened 9 years ago Closed 9 years ago

Set PollableEvent's mSignaled only if PR_Write succeeds

Categories

(Core :: Networking, defect)

Product:
Core
Component:
Networking
Version:
48 Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla49
Tracking Flags:
Tracking Status
firefox49 --- fixed

People

(Reporter: dragana, Assigned: dragana)

References

Details

(Whiteboard: [necko-active])

Attachments

(1 file)

bug_1270029.patch
830 bytes, patch
mcmanus
: review+
Details | Diff | Splinter Review 
      No description provided.
Assignee: nobody → dd.mozilla
Status: NEW → ASSIGNED
Whiteboard: [necko-active]

        Attachment #8748527 -
        Flags: review?(mcmanus)

    
  
Blocks: 1158189

    
    

    
  
Comment on attachment 8748527 [details] [diff] [review]
bug_1270029.patch

Review of attachment 8748527 [details] [diff] [review]:
-----------------------------------------------------------------

that must only happen as a shutdown race, right?

        Attachment #8748527 -
        Flags: review?(mcmanus) → review+

    
  
Keywords: checkin-needed

    
    

    
  
https://hg.mozilla.org/mozilla-central/rev/0e58a4abd21a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago

          status-firefox49:
          --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49

    
    

    
  
do we want to uplift this and ride the train with Bug 1158189?

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: