Crash in IconDatabase::retainIconForPageURL if icon database path is not writable

• Component: WebCore Misc.
• Status: NEW ---

[ian grant]

I'm new to running webkit and I am having problems launching the latest build on a ppc powerbook. I get 'safari has extensions installed' and a warning this may cause problems etc... 

I have removed the input managers that may be called extensions (sogudi etc.). and delete SIMBL directories in ~/Application Support/SIMBL etc. Still no application launch. Subsequent launches display a message like 'webkit had problems launching last time etc...' and fails to launch. I get the same warning about extensions on an intel laptop but have not installed any (I think). Webkit runs fine on the intel though after the initial warning. When I started running webkit - about 2 weeks ago - it ran fine on the PPC.

Mac os 10.4.7 (PPC)
The safari crash log is posted to the URL below: I will also attach it to this bug-report.

http://paste.lisp.org/display/24876

[ian grant]

Created attachment 10269 [details]
Safari Crash Log

Safari Crash Log

[Mark Rowe (bdash)]

The warning about extensions is a symptom of the crash during launch.  It will only appear if you have one or more extension installed, though it doesn't seem to be relevant based on your crash log.

Brady, the crash report indicates that it is failing during bindText16 inside IconDatabase::retainIconForPageURL so it seems likely that you'll be interested in this.

I cannot reproduce this on my Intel or PowerPC machines.

[ian grant]

Fixed - I had used 'Cocktail' http://www.maintain.se/cocktail/ 
to switch 'Do Not Cache Website Icons' preference of Safari on. I switched it off and webkit launched.

[Mark Rowe (bdash)]

Wow.  Cocktail disables the WebKit icon database by doing "chmod 500 ~/Library/Safari/Icons".  I would have thought that "defaults write com.apple.Safari WebIconDatabaseEnabled -bool NO" would have been more sensible, but what would I know.

I can reproduce this crash on both Intel and PowerPC Mac if I do: "chmod u= ~/Library/Safari/Icons".

[Brady Eidson]

This is disturbing.  
T'is why foreign binaries injected into webkit/safari aren't supported...

how'd you figure that out, Mark?

And yes, I will be interested, thanks for letting me know.  We obviously shouldn't crash on missing/inaccessible icon.db

Thanks

[Mark Rowe (bdash)]

I'm not sure I follow the comment about foreign binaries injected into Safari -- Cocktail is a standalone application for twiddling "settings" like that.  I tracked this one down the old-fashioned way -- gdb and ktrace on Cocktail while its changing the setting.

[Brady Eidson]

Nevermind then - I skimmed through and thought that Cocktail was an "extension" that ran inside the app.

Thanks  :)

[Brady Eidson]

<rdar://problem/4707718>

[Brady Eidson]

And now we're getting dupes of this issue -
http://bugzilla.opendarwin.org/show_bug.cgi?id=10630

A relatively simple fix is in the works

[Benjamin Morin]

*** Bug 10630 has been marked as a duplicate of this bug. ***

[Brady Eidson]

This has been fixed as of r16132 - please confirm and close