Code review comment for lp:~ochosi/indicator-power/xfce4-powermanager-settings

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for the work there, that seems slightly buggy though, in the case where the session is not Unity you do

" gchar *path
...
 g_free (path);"

that's likely to lead to invalid frees since you don't init path to NULL

review: Needs Fixing

« Back to merge proposal