[webkit-reviews] review denied: [Bug 204618] [GTK][WebInspector] Support for saving remote data : [Attachment 384351] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Nov 26 07:24:02 PST 2019

Carlos Garcia Campos <cgarcia at igalia.com> has denied Philippe Normand
<pnormand at igalia.com>'s request for review:
Bug 204618: [GTK][WebInspector] Support for saving remote data

Attachment 384351: Patch


--- Comment #2 from Carlos Garcia Campos <cgarcia at igalia.com> ---
Comment on attachment 384351
  --> https://bugs.webkit.org/attachment.cgi?id=384351

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

> Source/WebKit/UIProcess/gtk/RemoteWebInspectorProxyGtk.cpp:127
> +	   data = out.data();

You can't do this, out is going to be freed after the if, so data will point to
freed data.

> Source/WebKit/UIProcess/gtk/RemoteWebInspectorProxyGtk.cpp:130
> +	   data = content.utf8().data();

More ore less the same here, because the CString returned by utf8() is a

> Source/WebKit/UIProcess/gtk/RemoteWebInspectorProxyGtk.cpp:131
> +	   dataLength = content.utf8().length();

And we don't want to convert to utf8 twice either.

> Source/WebKit/UIProcess/gtk/RemoteWebInspectorProxyGtk.cpp:137
> +    if (g_file_set_contents(path.get(), data, dataLength, nullptr))
> +	  
t()), m_inspectorPage->webPageID());

You can use g_file_replace_contents_async(), instead of doing this

More information about the webkit-reviews mailing list