gcmd-devel
[Top][All Lists]
Advanced

[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.

Attachment: gnome-cmd-plain-path.cc.diff.tar.gz
Description: 1075582260-gnome-cmd-plain-path.cc.diff.tar.gz


reply via email to

[Prev in Thread] Current Thread [Next in Thread]