After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 316177 - "Invalid filename" error doesn't include filename
"Invalid filename" error doesn't include filename
Status: RESOLVED FIXED
Product: nautilus-cd-burner
Classification: Deprecated
Component: cd-burner
2.12.x
Other Linux
: Normal normal
: ---
Assigned To: Nautilus CD Burner Maintainers
Nautilus CD Burner Maintainers
: 322318 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-09-13 11:56 UTC by Sebastien Bacher
Modified: 2006-04-20 07:46 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12



Description Sebastien Bacher 2005-09-13 11:56:08 UTC
This bug has been opened here: http://bugzilla.ubuntu.com/show_bug.cgi?id=15321

"When trying to burn a cd with nautilus, I got a told it couldn't due to an
invalid filename. It would be good to get told which file was the problem, so
the I could rename it."
Comment 1 William Jon McCann 2005-09-29 15:07:07 UTC
This error is reported by mkisofs.  Unfortunately, it only reports: "Some files
have invalid filenames."
Comment 2 William Jon McCann 2005-11-28 17:10:31 UTC
*** Bug 322318 has been marked as a duplicate of this bug. ***
Comment 3 Cameron Patrick 2006-02-11 10:02:14 UTC
Mkisofs sometimes gives useful errors (version 2.01 in Ubuntu); e.g.

mkisofs: Error: [name omitted] and [name omitted] have the same Joliet name
mkisofs: Joliet tree sort failed. The -joliet-long switch may help you.
Comment 4 Mads Chr. Olesen 2006-02-11 19:59:57 UTC
http://bugzilla.gnome.org/attachment.cgi?id=55179&action=view is a patch tested on Ubuntu 5.10 (mkisofs version 2.01) to mention the offending filename in the dialog, as outputted by mkisofs.
Comment 5 Daniel Holbach 2006-04-07 18:13:52 UTC
Did anybody have a look at the patch yet?
Comment 6 William Jon McCann 2006-04-19 16:01:10 UTC
Ah.  Now I understand my confusion.  This error message isn't present in upstream cdrtools.  It is present in the de-facto "freedrtools" [1] fork:
http://cvs.fedora.redhat.com/viewcvs/devel/cdrtools/mkisofs-iconv-14.patch?view=markup

And it indeed includes the string/filename:

+               if(iconv(inls->iconv_d, &inptr, &inleft, &outptr, &outleft) ==
+                               (size_t)-1 && errno == EILSEQ) {
+                       fprintf(stderr, "Incorrectly encoded string (%s) "
+                               "encountered.\nPossibly creating an invalid "
+                               "Joliet extension. Aborting.\n", source);
+                       exit(1);
+               }
Comment 7 William Jon McCann 2006-04-19 16:18:26 UTC
I've committed a slightly modified version of your patch to HEAD.  The committed version fixed leaking the GString, doesn't use C++ comments, only g_print()'d  when in debugging, and followed the style (ie spaces before "(").

Please give this a try and let me know if there are any problems.  I haven't been able to test this because I can't figure out how to make an invalid filename.
Comment 8 John Thacker 2006-04-19 19:06:23 UTC
From http://www.portfoliofaq.com/pfaq/FAQ00352.htm

"#4. The 'Joliet' standard for Windows CD supporting long filenames has a limit of 64 characters for the total path (folders & filename). If you need deep nesting, use short folder and file names! Also watch for limitations on the depth of folder nesting that can occur with strict observance of ISO9660 (see first bullet in next list)."

That page may tell you how to make an invalid filename.
Comment 9 William Jon McCann 2006-04-19 19:09:49 UTC
John, I don't think this bug covers that kind of invalid filename.  This covers errors from iconv encoding.
Comment 10 James "Doc" Livingston 2006-04-20 07:46:47 UTC
I was the OP, but I'm not sure how I managed to produce an invalid file name. I haven't managed to see the error since.