bugCompact Disc Input and Control Library - Bugs: bug #39354, Bug in iso-read for ISO9660 images...

 
 

bug #39354: Bug in iso-read for ISO9660 images in 0.90 but not in 0.81

Submitted by:  Leo Baschy <infrastrug>
Submitted on:  Fri Jun 28 02:35:52 2013  
 
Category: run-time errorSeverity: 5 - Average
Item Group: libiso9660Status: None
Privacy: PublicAssigned to: Rocky Bernstein <rocky>
Open/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Mon Sep 2 06:58:46 2013, comment #15:

Am confirming this bug tests as fixed in c8d044e (as well as cb14b08) in 32 bit Enterprise Linux 6.4. Thank you for guidance at first and for integration of fix and test.

Leo Baschy <infrastrug>
Sun Sep 1 23:56:18 2013, comment #14:

Let me retract my last comment. This change DOES work for me.

The change was made in git commit c8d044e.

What remains is adapt iso-read-test.sh so it can be included in the normal tests.

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Sun Sep 1 23:18:26 2013, comment #13:

Thanks, Mike. I've changed the default but that doesn't address this bug for me. Did it for you? Or you Leo?

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Wed Aug 28 20:40:39 2013, comment #12:

change the 4th arg to AC_ARG_ENABLE(rock) to "yes". the code atm sets rock to disabled by default, but the intention is to enable it by default.

- enable_rock=$enableval, enable_rock=no)
+ enable_rock=$enableval, enable_rock=yes)

Mike Frysinger <vapier>
Tue Aug 20 00:43:05 2013, comment #11:

Compare iso-info output with iso-info --no-rock-ridge output.

And heck config.h to see HAVE_ROCK is #defined (to 1).

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Mon Aug 19 20:31:56 2013, comment #10:

Is there a simple (command line) way to tell whether the specific version of iso-read (or libcdio) one is looking at has been compiled with or without Rock Ridge support?

An answer might help find the cause faster.

Leo Baschy <infrastrug>
Mon Aug 19 03:33:19 2013, comment #9:

Thanks for the pinpointing this.

It looks like things go awry with Patch #7129 from Mike Frysinger: https://savannah.gnu.org/patch/?7129

The most obvious change is to try to properly handle ISO9660 without Rock-Ridge extensions and possibly those with Rock-Ridge extensions got messed up.

I spent a little time trying to pinpoint which part of the patch has problems to no avail. I don't have much free time to spend on this, so to the extent others can pitch in that would be appreciated.

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Wed Aug 14 01:08:36 2013, comment #8:

Without detailed description of the lengthy process and repeated attempts, here is what seems to be a reproducible result of bisection:

or

because the earlier of these two commit doesn't build.

Test done on 32-bit Enterprise Linux 6.4.

In addition to script from 39373 also used the following bisect-helper.sh made for this purpose:

Tried various ranges from 0.81 to HEAD.

Leo Baschy <infrastrug>
Fri Aug 9 15:29:56 2013, comment #7:

Thanks for the info.

Now what is needed is to use that in a git bisect to find a commit which caused this to fail.

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Fri Aug 9 15:16:25 2013, comment #6:

For fixing this regression bug 39354 if you use the test script from 39373 for now (before 39373 is fixed) you should only look at the 3G size .iso file results (less than 4G size where bug 39373 kicks in).

Leo Baschy <infrastrug>
Tue Jul 30 13:30:43 2013, comment #5:

Thank you for creating this test. There's one more thing that with this test that will greatly speed up fixing this bug.

If you use that test in "git bisect run" (see http://git-scm.com/book/en/Git-Tools-Debugging-with-Git#Binary-Search ) to narrow the problem down to a commit, I'll make sure this gets fixed faster because I can probably enlist the help and advice of those who made the change. Thanks

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Mon Jul 29 19:49:40 2013, comment #4:

Now there is a test script at bug 39373.

Leo Baschy <infrastrug>
Sun Jun 30 02:20:56 2013, comment #3:

Maybe I wasn't clear enough in comment #1, yes I've verified the defect in Scientific Linux 6.4 (as a host too, not just as sample .iso file).

That means same should be in RHEL 6.4 and CentOS 6.4. They come with 0.81 and it works. Then I built 0.90 from sources and installed it and the defect showed. Then I built 0.81 from sources this time, no defect again. Reproducibly 0.90 is broken versus 0.81 works.

So if you have a RHEL 6.x or derivative, grab the .iso I've linked and see defect with 2/3rd of 3000+ files e.g.

and interestingly no defect with 1/3 of 3000+ files e.g.

Leo Baschy <infrastrug>
Sat Jun 29 17:29:49 2013, comment #2:

I don't have a Mac or access to OSX. Does this fail on an OS other than OSX?

Rocky Bernstein <rocky>
Project AdministratorIn charge of this item.
Fri Jun 28 19:30:31 2013, comment #1:

Took 0.90 sources and built and installed in Enterprise Linux 6.4 and it also fails for same file Packages/ConsoleKit-libs-0.4.1-3.el6.i686.rpm from that same .iso file.

It did work minutes earlier on that machine with 0.81.

This looks like a regression, any platform. Not Mac specific.

There was a typo in original post, but to be clear, it is the ConsoleKit-libs file that cannot be read from that .iso file (and 2/3rd of the files on that .iso file cannot be read).

And it does work for the remaining 1/3rd of the files on that .iso file, and in 0.81 it used to work for all of them.

That .iso file is a free download of a Linux distro, linked in original post, so you can try it out.

Leo Baschy <infrastrug>
Fri Jun 28 02:35:52 2013, original submission:

Using ISO file SL-64-i386-2013-03-18-Install-DVD.iso with NrvrCommander, which for some feature uses iso-read.

Works fine with iso-read version 0.81 x86_64-koji-linux-gnu coming with RHEL 6.3 and 6.4.

Fails with iso-read version 0.90 x86_64-apple-darwin12.2.0 from MacPorts in OS X 10.8.

Version 0.90 build on Mac fails (but 0.81 in Linux succeeds) on 2221 of 3104 files in that .iso. For example it fails at

Fails with error

For example, however, it succeeds in same .iso at

Version 0.81 in RHEL flavor Linux succeeds on all files in that .iso file.

Leo Baschy <infrastrug>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #28838:  bisect-helper.sh added by infrastrug (864B - application/octet-stream - bisect-helper.sh script use with iso-read-test.sh, worked at least 0.81 to 0.90)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by vapier (Posted a comment)
  • -unavailable- added by rocky (Posted a comment)
  • -unavailable- added by infrastrug (Submitted the item)
  • -unavailable- added by infrastrug
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Sep 2 09:04:09 2013rockyOpen/ClosedOpen=>Closed
    Mon Aug 19 03:33:19 2013rockyAssigned toNone=>rocky
    Wed Aug 14 01:19:25 2013infrastrugAttached File-=>Added bisect-helper.sh, #28838
    Wed Aug 7 05:04:57 2013rockySummaryBug in handling ISO9660 images over 4G in 0.90 but not 0.81=>Bug in iso-read for ISO9660 images in 0.90 but not in 0.81
    Tue Jul 30 18:33:41 2013rockyItem Groupiso-read=>libiso9660
      Summaryiso-read 0.90 on Mac fails for many but not for all files in .iso=>Bug in handling ISO9660 images over 4G in 0.90 but not 0.81
    Fri Jun 28 02:35:52 2013infrastrugCarbon-Copy-=>Added infrastrug

    Back to the top


    Powered by Savane 3.1-cleanup