gvfs: Add an assertion to check that get_file_for_uri() is never NULL

It cannot return a NULL value, as none of its callers have error
handlng. Add an assertion to check the values returned by the VFS
implementations.

Signed-off-by: Philip Withnall <withnall@endlessm.com>

Helps: #1819
This commit is contained in:
Philip Withnall 2019-07-02 12:08:45 +01:00
parent 29388470f8
commit e24503a8e9

View File

@ -236,7 +236,7 @@ g_vfs_get_file_for_uri (GVfs *vfs,
const char *uri)
{
GVfsClass *class;
GFile *ret;
GFile *ret = NULL;
g_return_val_if_fail (G_IS_VFS (vfs), NULL);
g_return_val_if_fail (uri != NULL, NULL);
@ -244,10 +244,12 @@ g_vfs_get_file_for_uri (GVfs *vfs,
class = G_VFS_GET_CLASS (vfs);
ret = get_file_for_uri_internal (vfs, uri);
if (ret)
return ret;
if (!ret)
ret = (* class->get_file_for_uri) (vfs, uri);
return (* class->get_file_for_uri) (vfs, uri);
g_assert (ret != NULL);
return g_steal_pointer (&ret);
}
/**