Closed
      
        Bug 1256181
      
      
        Opened 9 years ago
          Closed 9 years ago
      
        
    
  
Various tests use a function getContentFromMessage() and don't ever close the input stream.    
    Categories
(MailNews Core :: Testing Infrastructure, defect)
        MailNews Core
          
        
        
      
        
    
        Testing Infrastructure
          
        
        
      
        
    Tracking
(firefox48 affected)
        RESOLVED
        FIXED
        
    
  
        
            Thunderbird 48.0
        
    
  
| Tracking | Status | |
|---|---|---|
| firefox48 | --- | affected | 
People
(Reporter: jorgk-bmo, Assigned: jorgk-bmo)
Details
Attachments
(1 file)
| 
        
        
         4.70 KB,
          patch         
       | 
      
           mkmelin
 :
              
              review+
           | 
      Details | Diff | Splinter Review | 
mail/test/mozmill/composition/test-charset-upgrade.js
mailnews/base/test/unit/test_detachToFile.js
mailnews/base/test/unit/test_mimemaltdetach.js
mailnews/base/test/unit/test_quarantineFilterMove.js
mailnews/imap/test/unit/test_imapAttachmentSaves.js
need fixing.
| Assignee | ||
          Comment 1•9 years ago
           
         | 
      ||
OK, little boyscout clean-up as requested ;-)
Assignee: nobody → mozilla
Status: NEW → ASSIGNED
        Attachment #8730024 -
        Flags: review?(mkmelin+mozilla)
          Comment 2•9 years ago
           
         | 
      ||
Comment on attachment 8730024 [details] [diff] [review]
This should do it.
Review of attachment 8730024 [details] [diff] [review]:
-----------------------------------------------------------------
Thx, r=mkmelin
        Attachment #8730024 -
        Flags: review?(mkmelin+mozilla) → review+
| Assignee | ||
          Updated•9 years ago
           
         | 
      
Keywords: checkin-needed
          Comment 3•9 years ago
           
         | 
      ||
https://hg.mozilla.org/comm-central/rev/f13c6a6f45b1a010516ce3ddfa8c4e5abb396c58
Bug 1256181 - Close input stream in getContentFromMessage(). r=mkmelin
          Updated•9 years ago
           
         | 
      
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 48.0
          You need to log in
          before you can comment on or make changes to this bug.
        
Description
•