Code review comment for lp:~alecu/ubuntuone-client/fix-nautilus-symlinks-udfs

Revision history for this message
dobey (dobey) wrote :

+ gfile = g_file_new_for_uri (nautilus_file_info_get_uri (file));

This will leak memory. You need to assign the result of nautilus_file_info_get_uri () to a gchar *, and then g_free () it when done using it, as it does not return a const. I think.

review: Needs Fixing

« Back to merge proposal