WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
115176
Web Inspector: Crash due to null items from getDOMStorageItems
https://bugs.webkit.org/show_bug.cgi?id=115176
Summary
Web Inspector: Crash due to null items from getDOMStorageItems
Konrad Piascik
Reported
2013-04-25 06:42:11 PDT
Web Inspector: Crash due to null items from getDOMStorageItems
Attachments
Patch
(5.78 KB, patch)
2013-04-26 06:11 PDT
,
Konrad Piascik
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Konrad Piascik
Comment 1
2013-04-25 07:22:47 PDT
This asserts in DEBUG builds as well 0x7cb22ec4 in WebCore::InspectorObjectBase::setValue (this=0x774bfcf8, name=..., value=...) at /home/kpiascik/dev/webkit/Source/WebCore/inspector/InspectorValues.h:336 336 ASSERT(value); (gdb) bt #0 0x7cb22ec4 in WebCore::InspectorObjectBase::setValue (this=0x774bfcf8, name=..., value=...) at /home/kpiascik/dev/webkit/Source/WebCore/inspector/InspectorValues.h:336 #1 0x7d8e37f8 in WebCore::InspectorBackendDispatcherImpl::DOMStorage_getDOMStorageItems (this=0x7a046518, callId=62, requestMessageObject=0x7760f0e0) at /home/kpiascik/dev/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/InspectorBackendDispatcher.cpp:2555 #2 0x7d8fb6cc in WebCore::InspectorBackendDispatcherImpl::dispatch (this=0x7a046518, message=...) at /home/kpiascik/dev/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/InspectorBackendDispatcher.cpp:6650 #3 0x7cba44ec in WebCore::InspectorController::dispatchMessageFromFrontend (this=0x799c8b48, message=...) at /home/kpiascik/dev/webkit/Source/WebCore/inspector/InspectorController.cpp:360 (gdb) up #1 0x7d8e37f8 in WebCore::InspectorBackendDispatcherImpl::DOMStorage_getDOMStorageItems (this=0x7a046518, callId=62, requestMessageObject=0x7760f0e0) at /home/kpiascik/dev/webkit/WebKitBuild/armle-v7/Debug/DerivedSources/WebCore/InspectorBackendDispatcher.cpp:2555 2555 result->setValue("entries", out_entries); (gdb) p out_entries $1 = {m_ptr = 0x0}
Konrad Piascik
Comment 2
2013-04-26 06:11:15 PDT
Created
attachment 199821
[details]
Patch
Joseph Pecoraro
Comment 3
2013-04-26 11:13:08 PDT
Comment on
attachment 199821
[details]
Patch Nice, good test.
WebKit Commit Bot
Comment 4
2013-04-26 11:56:34 PDT
Comment on
attachment 199821
[details]
Patch Clearing flags on attachment: 199821 Committed
r149200
: <
http://trac.webkit.org/changeset/149200
>
WebKit Commit Bot
Comment 5
2013-04-26 11:56:36 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug