My favorites ▼ | Sign in
Logo
Project Home
  
Downloads
  
Wiki
  
Issues
    
New issue | Search
for
| Advanced search | Search tips
Issue 18140: Crash - SkBitmap::operator=(SkBitmap const &)
1 person starred this issue and may be notified of changes. Back to list
 
Reported by crash...@chromium.org, Jul 30, 2009 
This crash was detected in 3.0.196.0 and appears to be a regression from 3.0.195.4.
It is currently ranked #1 (based on the relative number of reports in the release).  There have been 80 reports from 22 clients.
Search query: http://crash/search?query=Chrome+3.0.196.0+SkBitmap%3A%3Aoperator%3D%28SkBitmap+const+%26%29
----------------------------
*       Summary Data       *
----------------------------
Report Link: http://crash/reportdetail?reportid=927d4aef85fc8e03
Mini Dump Link: http://crash/file?reportid=927d4aef85fc8e03&name=upload_file_minidump

Uptime: 6 sec
User Comments: null
OS: Windows Vista or Windows Server 2008 Service Pack 2
CPU Architecture: x86
CPU Info: GenuineIntel family 6 model 23 stepping 10
rept: null
ptype: browser
plat: Win32
crash type:(EXCEPTION_ACCESS_VIOLATION@0x00000000)

----------------------------
*        Crash Trace       *
----------------------------
          [skbitmap.cpp:103] - SkBitmap::operator=(SkBitmap const &)
 [location_bar_view.cc:1321] - LocationBarView::PageActionImageView::OnImageLoaded(SkBitmap *,unsigned int)
 [location_bar_view.cc:1158] - LocationBarView::PageActionImageView::ImageLoadingTracker::OnImageLoaded(SkBitmap *,int)
                [task.h:307] - RunnableMethod<HistogramSynchronizer,void ( HistogramSynchronizer::*)(MessageLoop *,Task *),Tuple2<MessageLoop *,Task *> >::Run()
       [message_loop.cc:313] - MessageLoop::RunTask(Task *)
       [message_loop.cc:321] - MessageLoop::DeferOrRunPendingTask(MessageLoop::PendingTask const &)
       [message_loop.cc:428] - MessageLoop::DoWork()
   [message_pump_win.cc:209] - base::MessagePumpForUI::DoRunLoop()
    [message_pump_win.cc:52] - base::MessagePumpWin::RunWithDispatcher(base::MessagePump::Delegate *,base::MessagePumpWin::Dispatcher *)
       [message_loop.cc:193] - MessageLoop::RunInternal()
       [message_loop.cc:181] - MessageLoop::RunHandler()
       [message_loop.cc:590] - MessageLoopForUI::Run(base::MessagePumpWin::Dispatcher *)
       [browser_main.cc:195] - `anonymous namespace'::RunUIMessageLoop(BrowserProcess *)
       [browser_main.cc:786] - BrowserMain(MainFunctionParams const &)
    [chrome_dll_main.cc:539] - ChromeMain
[google_update_client.cc:96] - google_update::GoogleUpdateClient::Launch(HINSTANCE__ *,sandbox::SandboxInterfaceInfo *,wchar_t *,char const *,int *)
   [kernel32.dll+0x000affff] - GetMBCompMB

----------------------------
*      Loaded Modules      *
----------------------------
    GoogleDesktopAPI2.dll
    GoogleDesktopCommon.dll
    GoogleDesktopNetwork3.dll
    GoogleDesktopResources_en.dll
    chrome.dll
    gears.dll
    icudt38.dll
    chrome.exe
    Wldap32.dll
    advapi32.dll
    clbcatq.dll
    gdi32.dll
    iertutil.dll
    kernel32.dll
    lpk.dll
    msctf.dll
    msvcrt.dll
    normaliz.dll
    nsi.dll
    ntdll.dll
    ole32.dll
    oleaut32.dll
    psapi.dll
    rpcrt4.dll
    schannel.dll
    secur32.dll
    setupapi.dll
    shell32.dll
    shlwapi.dll
    urlmon.dll
    user32.dll
    usp10.dll
    wininet.dll
    ws2_32.dll
    IPHLPAPI.DLL
    NapiNSP.dll
    WSHTCPIP.DLL
    apphelp.dll
    credssp.dll
    crypt32.dll
    dhcpcsvc.dll
    dhcpcsvc6.DLL
    dnsapi.dll
    dwmapi.dll
    imm32.dll
    msasn1.dll
    mswsock.dll
    netapi32.dll
    nlaapi.dll
    ntmarta.dll
    oleacc.dll
    pnrpnsp.dll
    rasadhlp.dll
    riched20.dll
    samlib.dll
    sxs.dll
    t2embed.dll
    userenv.dll
    uxtheme.dll
    version.dll
    winhttp.dll
    winmm.dll
    winnsi.dll
    winrnr.dll
    wshbth.dll
    wship6.dll
    comctl32.dll
Comment 1 by laforge@chromium.org, Jul 30, 2009 
(No comment was entered for this change.)
Status: Assigned
Owner: fin...@chromium.org
Cc: r...@android.com g...@chromium.org
Comment 2 by glen@chromium.org, Jul 30, 2009 
I think Finnur's out on vacation; not sure he'd have checked anything in between 195.4 
and 196. I'd take it, but I have negative time until Saturday. Assigning to Miranda in the 
hope she can get it done tomorrow (sorry Miranda - please reassign if you feel it's 
necessary).
Owner: miran...@chromium.org
Cc: lafo...@chromium.org fin...@chromium.org
Comment 3 by bugdroid1@chromium.org, Jul 31, 2009 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22213 

------------------------------------------------------------------------
r22213 | mirandac@chromium.org | 2009-07-31 15:58:40 -0700 (Fri, 31 Jul 2009) | 6 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/location_bar_view.cc?r1=22213&r2=22212

LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases.

BUG= http://crbug.com/18140
TEST= none.

Review URL: http://codereview.chromium.org/159711
------------------------------------------------------------------------
Comment 4 by bugdroid1@chromium.org, Aug 03, 2009 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22254 

------------------------------------------------------------------------
r22254 | mirandac@chromium.org | 2009-08-02 12:35:20 -0700 (Sun, 02 Aug 2009) | 8 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/location_bar_view.cc?r1=22254&r2=22253

LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases.

BUG= http://crbug.com/18140
TEST= none.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22213
Review URL: http://codereview.chromium.org/159711
------------------------------------------------------------------------
Comment 5 by bugdroid1@chromium.org, Aug 03, 2009 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22267 

------------------------------------------------------------------------
r22267 | mirandac@chromium.org | 2009-08-03 08:34:07 -0700 (Mon, 03 Aug 2009) | 10 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/location_bar_view.cc?r1=22267&r2=22266

LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases.

BUG= http://crbug.com/18140
TEST= none.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22213

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22254
Review URL: http://codereview.chromium.org/159711
------------------------------------------------------------------------
Comment 6 by mirandac@chromium.org, Aug 03, 2009 
(No comment was entered for this change.)
Status: Fixed
Comment 7 by bugdroid1@chromium.org, Aug 07, 2009 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22845 

------------------------------------------------------------------------
r22845 | laforge@chromium.org | 2009-08-07 17:54:54 -0700 (Fri, 07 Aug 2009) | 10 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/views/location_bar_view.cc?r1=22845&r2=22844

Merge 22213 - LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases.

BUG= http://crbug.com/18140
TEST= none.

Review URL: http://codereview.chromium.org/159711

TBR=mirandac@chromium.org

Review URL: http://codereview.chromium.org/165197
------------------------------------------------------------------------
Comment 8 by bugdroid1@chromium.org, Aug 07, 2009 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22847 

------------------------------------------------------------------------
r22847 | laforge@chromium.org | 2009-08-07 18:06:44 -0700 (Fri, 07 Aug 2009) | 12 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/views/location_bar_view.cc?r1=22847&r2=22846

Merge 22254 - LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases.

BUG= http://crbug.com/18140
TEST= none.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22213
Review URL: http://codereview.chromium.org/159711

TBR=mirandac@chromium.org

Review URL: http://codereview.chromium.org/165199
------------------------------------------------------------------------
Comment 9 by bugdroid1@chromium.org, Aug 07, 2009 
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=22849 

------------------------------------------------------------------------
r22849 | laforge@chromium.org | 2009-08-07 18:08:57 -0700 (Fri, 07 Aug 2009) | 14 lines
Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/views/location_bar_view.cc?r1=22849&r2=22848

Merge 22267 - LocationBarView::PageActionImageView::LoadImageTask::Run() may pass a NULL image pointer through to OnImageLoaded; make sure that the pointer is not dereferenced in these cases.

BUG= http://crbug.com/18140
TEST= none.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22213

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=22254
Review URL: http://codereview.chromium.org/159711

TBR=mirandac@chromium.org

Review URL: http://codereview.chromium.org/164220
------------------------------------------------------------------------
► Sign in to add a comment

Powered by Google Project Hosting