Closed Bug 297752 Opened 19 years ago Closed 19 years ago

Thunderbird -version core dump

Categories

(Thunderbird :: General, defect)

Product:
Thunderbird
Component:
General
Platform:
Sun
Solaris
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: leon.sha, Assigned: mscott)

Details

Attachments

(1 file, 2 obsolete files)

Patch v3
913 bytes, patch
darin.moz
: review+
asa
: approval-aviary1.1a2+
Details | Diff | Splinter Review 
Thunderbird -version core dump.
The stach is below.
 ff2ae9b0 strlen   (505b5, ffbfe54c, ffbfddc9, 0, 0, 0) + 50
 ff315660 printf   (505b3, ff36875c, ff36877a, ff3682e8, ff364224, 1000) + d8
 000247d4 __1cLDumpVersion6F_v_ (1, 0, 9260c, 66568, ff364224, ff36c8d0) + 8c
 00028978 __1cIxre_main6FippcpknMnsXREAppData__i_ (2, ffbfef34, 66e88, fed46f00,
fcb90200, fcb90240) + 2b8
 0002187c main     (2, ffbfef34, ffbfef40, 66c00, fcb90180, fcb901c0) + 44
 00021408 _start   (0, 0, 0, 0, 0, 0) + 108
Attached patch Patch (obsolete) — — Splinter Review 

        Attachment #186300 -
        Flags: review?(darin)

    
    

    
  
Comment on attachment 186300 [details] [diff] [review]
Patch

This has the effect of moving the thunderbird profile directory.  That means
that users will have to start over with a fresh profile.  That is no good. 
Moreover, the API for XRE_Main allows this field to be null, so we should make
sure that null works.

        Attachment #186300 -
        Flags: review?(darin) → review-

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Patch v2 (obsolete)
        — 
          — Splinter Review
      

    


  

        Attachment #186300 -
        Attachment is obsolete: true

        Attachment #186993 -
        Flags: review?(darin)

    
    

    
  
This patch will change firefox too. But I think the vendor is not needed for
firefox -version. Or we can add ifdef here.

    
    

    
  
Comment on attachment 186993 [details] [diff] [review]
Patch v2

I think the vendor string should be printed if non-null.

        Attachment #186993 -
        Flags: review?(darin) → review-

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Patch v3
        — 
          — Splinter Review
      

    


  

        Attachment #186993 -
        Attachment is obsolete: true

        Attachment #188517 -
        Flags: review?(darin)

    
  

        Attachment #188517 -
        Flags: review?(darin) → review+

    
  

        Attachment #188517 -
        Flags: approval-aviary1.1a2?

    
  

        Attachment #188517 -
        Flags: approval-aviary1.1a2? → approval-aviary1.1a2+

    
    

    
  
Checking in toolkit/xre/nsAppRunner.cpp;
/cvsroot/mozilla/toolkit/xre/nsAppRunner.cpp,v  <--  nsAppRunner.cpp
new revision: 1.100; previous revision: 1.99
done

Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: