Closed Bug 423843 Opened 12 years ago Closed 12 years ago

Subtile passes null to RDFServiceImpl::GetLiteral, triggering ###!!! ASSERTION: null ptr: 'aValue != nsnull'

Categories

(Firefox :: Extension Compatibility, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: cbook, Assigned: michel.gutierrez)

References

()

Details

(Keywords: assertion)

Attachments

(1 file)

Attached file stack
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9b5pre) Gecko/2008031718 Firefox/3.0b5pre ID:2008031718

Steps to reproduce:
-> Install the Subtile Extension https://addons.mozilla.org/en-US/firefox/addon/4906
-> Make this Extension compatible to your Debug Build (its a Beta 3 Extension)
-> Restart Firefox with Subtile Extension enabled
--> Assertion

###!!! ASSERTION: null ptr: 'aValue != nsnull', file /debug/mozilla/rdf/base/src/nsRDFService.cpp, line 1112
Is this extension using a binary component? If so, it's passing null for a string argument, which would be a bug in the extension.

If it isn't using a binary component, then what is the JS code that is being called when it crashes?
Tomcat, can you get a C++ stack trace for the assertion?
Never mind, timeless says the JS stack (DumpJSStack) is more useful for this bug than the C++ stack.
for reference
components/ilUtilService.js:109
        value=this.RDF.GetLiteral(value);

tomcat provided the correct stack. however it's a bug in the extension, so it should be in some other product.
Component: General → Extension Compatibility
Product: Core → Firefox
QA Contact: general → extension.compatibility
timeless agrees with the bug 421720 comment 7 doctrine ("abusing a scriptable API should not trigger assertions"), but thinks there's already a bug report about this assertion being bogus, so he decided that this bug report should be about fixing the extension.
Summary: ###!!! ASSERTION: null ptr: 'aValue != nsnull' → Subtile passes null to RDFServiceImpl::GetLiteral, triggering ###!!! ASSERTION: null ptr: 'aValue != nsnull'
All right, i see the problem.

I've made a fix, it is available at http://www.subtile.net/subtile-1.3b1.xpi

If someone can verify the error does not appear anymore, i'll submit version 1.3 of Subtile to addons.mozilla.org.
(In reply to comment #6)
> All right, i see the problem.
> 
> I've made a fix, it is available at http://www.subtile.net/subtile-1.3b1.xpi
> 
> If someone can verify the error does not appear anymore, i'll submit version
> 1.3 of Subtile to addons.mozilla.org.
> 

Hi Michel, i don't see this assertion with your 1.3b1 version of subtiles. So verified fixed. Thanks everyone for the great work here.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee: nobody → michel.gutierrez
You need to log in before you can comment on or make changes to this bug.