Crash under WebKit::LaunchServicesDatabaseManager::didConnect()

[Per Arne Vollan]

There is a crash under WebKit::LaunchServicesDatabaseManager::didConnect() when trying to send a message to an invalid connection.

[Per Arne Vollan]

Created attachment 404154 [details]
Patch

[Per Arne Vollan]

Created attachment 404155 [details]
Patch

[Darin Adler]

Comment on attachment 404155 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=404155&action=review

> Source/WebKit/WebProcess/cocoa/LaunchServicesDatabaseManager.mm:70
> +    auto conn = connection();

WebKit coding style says:

    auto connection = this->connection();

[Per Arne Vollan]

Created attachment 404161 [details]
Patch

[Per Arne Vollan]

(In reply to Darin Adler from comment #3)
> Comment on attachment 404155 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=404155&action=review
> 
> > Source/WebKit/WebProcess/cocoa/LaunchServicesDatabaseManager.mm:70
> > +    auto conn = connection();
> 
> WebKit coding style says:
> 
>     auto connection = this->connection();

Fixed.

Thanks for reviewing!

[EWS]

Committed r264309: <https://trac.webkit.org/changeset/264309>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 404161 [details].

[Radar WebKit Bug Importer]

<rdar://problem/65492080>

[Per Arne Vollan]

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

[Ryan Haddad]

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