Closed Bug 380975 Opened 17 years ago Closed 17 years ago

Branch null-ptr crashes in nsAccessibleHyperText (Linux only) [@ nsAccessibleHyperText::GetCaretOffset]

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Version:
1.8 Branch
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)

Details

(5 keywords)

Crash Data

Attachments

(2 files)

Branch 1.8 patch, rev. 1
(deleted), patch
aaronlev
: review+
dveditz
: approval1.8.1.5+
Details | Diff | Splinter Review
Branch 1.8.0 patch, rev. 1
(deleted), patch
aaronlev
: review+
dveditz
: approval1.8.0.13+
Details | Diff | Splinter Review 
Spawned off from bug 376924 comment 32 - the "Standalone testcase"
attachment 261377 [details], crashes on branches, also with the patch in bug 376924.
Version: Trunk → 1.8 Branch
Attached patch Branch 1.8 patch, rev. 1 (deleted) — — Splinter Review 

        Attachment #265084 -
        Flags: review?(aaronleventhal)

    
    

    
  

      
      
      
      

        Attached patch
          
          
        Branch 1.8.0 patch, rev. 1 (deleted)
        — 
          — Splinter Review
      

    


  

        Attachment #265085 -
        Flags: review?(aaronleventhal)

    
    

    
  
These patches + the one in bug 376924 fixes the crash on the testcase...

    
  
Severity: normal → critical
Keywords: crash

    
  

        Attachment #265084 -
        Flags: review?(aaronleventhal) → review+

    
  

        Attachment #265085 -
        Flags: review?(aaronleventhal) → review+

    
  

        Attachment #265084 -
        Flags: approval1.8.1.5?

    
  

        Attachment #265085 -
        Flags: approval1.8.0.13?

    
    

    
  
Comment on attachment 265084 [details] [diff] [review]
Branch 1.8 patch, rev. 1

approved for 1.8.1.5 and 1.8.0.13, a=dveditz for release-drivers

        Attachment #265084 -
        Flags: approval1.8.1.5? → approval1.8.1.5+

        Attachment #265085 -
        Flags: approval1.8.0.13? → approval1.8.0.13+

    
    

    
  
Comment on attachment 265084 [details] [diff] [review]
Branch 1.8 patch, rev. 1

Checked in to MOZILLA_1_8_BRANCH:
accessible/src/atk/nsAccessibleHyperText.cpp:  1.25.8.4

    
    

    
  
Comment on attachment 265085 [details] [diff] [review]
Branch 1.8.0 patch, rev. 1

Checked in to MOZILLA_1_8_0_BRANCH:
accessible/src/atk/nsAccessibleHyperText.cpp:  1.25.12.1

    
  
Status: NEW → RESOLVED
Closed: 17 years ago
Keywords: fixed1.8.0.13, 
          
            fixed1.8.1.5
Resolution: --- → FIXED

    
    

    
  
Ff the <html> or <body> had aaa:describedby there could be a description for the element. See GetRoleContent().

    
    

    
  
Sorry, I meant that comment for a different bug.

    
    

    
  
verified fixed 1.8.1.5 using the testcase https://bugzilla.mozilla.org/attachment.cgi?id=261377 and Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.5pre) Gecko/2007071003 BonEcho/2.0.0.5pre on Fedora F7  - no crash - adding verified keyword
Keywords: fixed1.8.1.5 → verified1.8.1.5

    
    

    
  
verified fixed 1.8.0.13 using Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.13pre) Gecko/20070810 Firefox/1.5.0.13pre on Fedora F7 - no crash on testcase - adding verified keyword
Keywords: fixed1.8.0.13 → verified1.8.0.13
Crash Signature: [@ nsAccessibleHyperText::GetCaretOffset]

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: