[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [gcmd-dev] fix for bug #360175
From: |
Koos Jan Niesink |
Subject: |
Re: [gcmd-dev] fix for bug #360175 |
Date: |
Tue, 24 Apr 2007 08:57:28 +0100 (BST) |
Hello again,
I forgot the unref the GnomeVFSURI *t, so here is an update.
On a side note. An other solution to the fix would be the following; leave out
the GnomeVFSURI *t and replace it with
u1 = gnome_vfs_new (gnome_vfs_make_uri_from_input (GNOME_CMD_PLAIN_PATH
(path)->priv->path));
Maybe the latter needs less memory. Anyway, both solutions work.
koos
Michael <address@hidden> wrote: Koos,
> Did any of you experience this bug as well? To reproduce this one. First
> create a dir with a %-sign in it's name. Then cd to that dir and create
> another file or dir in it.
It says 'file exists' no matter what i try. I can work via commandline,
normally, however.
But then, afterwards, reloading the file list crashs.
Let's see how your fix is working :)
> This bug was caused by the fact that the function 'gnome_vfs_uri_new' somehow
> didn't create a valid uri.
% \ & > / all these are critical chars in a filename. One would not use them,
normally, but they can be misused, e.g. in malicious filenames of website
downloads or email attachments.
A filemanager need to be immune, of course. So, you've done a good job :)
> ps. do any of you experience the rename bug as well?
what was it, again ?
m°
_______________________________________________
gcmd-devel mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/gcmd-devel
---------------------------------
Yahoo! Mail is the world's favourite email. Don't settle for less, sign up for
your freeaccount today.
gnome-cmd-plain-path.cc.diff.tar.gz
Description: 1075582260-gnome-cmd-plain-path.cc.diff.tar.gz