Your message dated Fri, 08 Sep 2006 06:02:05 -0700
with message-id <[EMAIL PROTECTED]>
and subject line Bug#261274: fixed in bchunk 1.2.0-3
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--- Begin Message ---
Package: bchunk
Version: 1.2.0-1
Severity: important
Tags: patch



-- System Information:
Debian Release: testing/unstable
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)
Kernel: Linux 2.6.7
Locale: LANG=pl_PL.UTF-8, LC_CTYPE=pl_PL.UTF-8

Versions of packages bchunk depends on:
ii  libc6                       2.3.2.ds1-13 GNU C Library: Shared libraries an

-- no debconf information


If the cue file specifies several tracks than bchunk incorrectly calculates the 
size of the tracks which leads to an off-by-one error (one sector, i.e. 2352 
bytes). The attached patch solves this problem.

Piotr Kaczuba
--- bchunk-1.2.0/bchunk.c	2004-06-29 22:42:33.000000000 +0200
+++ bchunk-1.2.0.new/bchunk.c	2004-07-24 17:52:05.000000000 +0200
@@ -289,12 +289,12 @@
 		exit(4);
 	}
 	
-	reallen = (track->stopsect - track->startsect + 1) * track->bsize;
+	reallen = (track->stopsect - track->startsect) * track->bsize;
 	if (verbose) {
-		printf("\n mmc sectors %ld->%ld (%ld)", track->startsect, track->stopsect, track->stopsect - track->startsect + 1);
-		printf("\n mmc bytes %ld->%ld (%ld)", track->start, track->stop, track->stop - track->start + 1);
+		printf("\n mmc sectors %ld->%ld (%ld)", track->startsect, track->stopsect - 1, track->stopsect - track->startsect);
+		printf("\n mmc bytes %ld->%ld (%ld)", track->start, track->stop - 1, track->stop - track->start);
 		printf("\n sector data at %d, %d bytes per sector", track->bstart, track->bsize);
-		printf("\n real data %ld bytes", (track->stopsect - track->startsect + 1) * track->bsize);
+		printf("\n real data %ld bytes", (track->stopsect - track->startsect) * track->bsize);
 		printf("\n");
 	}
 
@@ -332,7 +332,7 @@
 	sz = track->start;
 	sect = track->startsect;
 	fl = 0;
-	while ((sect <= track->stopsect) && (fread(buf, SECTLEN, 1, bf) > 0)) {
+	while ((sect < track->stopsect) && (fread(buf, SECTLEN, 1, bf) > 0)) {
 		if (track->audio) {
 			if (swabaudio) {
 				/* swap low and high bytes */
@@ -477,7 +477,7 @@
 				printf(" (startsect %ld ofs %ld)", track->startsect, track->start);
 			if ((prevtrack) && (prevtrack->stopsect < 0)) {
 				prevtrack->stopsect = track->startsect;
-				prevtrack->stop = track->start - 1;
+				prevtrack->stop = track->start;
 			}
 		}
 	}

--- End Message ---
--- Begin Message ---
Source: bchunk
Source-Version: 1.2.0-3

We believe that the bug you reported is fixed in the latest version of
bchunk, which is due to be installed in the Debian FTP archive:

bchunk_1.2.0-3.diff.gz
  to pool/main/b/bchunk/bchunk_1.2.0-3.diff.gz
bchunk_1.2.0-3.dsc
  to pool/main/b/bchunk/bchunk_1.2.0-3.dsc
bchunk_1.2.0-3_i386.deb
  to pool/main/b/bchunk/bchunk_1.2.0-3_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [EMAIL PROTECTED],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Daniel Baumann <[EMAIL PROTECTED]> (supplier of updated bchunk package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [EMAIL PROTECTED])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Fri,  8 Sep 2006 14:45:00 +0200
Source: bchunk
Binary: bchunk
Architecture: source i386
Version: 1.2.0-3
Distribution: unstable
Urgency: low
Maintainer: Daniel Baumann <[EMAIL PROTECTED]>
Changed-By: Daniel Baumann <[EMAIL PROTECTED]>
Description: 
 bchunk     - CD image format conversion from bin/cue to iso/cdr
Closes: 261274
Changes: 
 bchunk (1.2.0-3) unstable; urgency=low
 .
   * Added 01-track-size.dpatch from Piotr Kaczuba <[EMAIL PROTECTED]> to fix
     wrong track size calculation when having multiple tracks in one image
     (Closes: #261274).
Files: 
 5c70abd077b6cc5f7dcafc88fc75fd6c 561 otherosfs optional bchunk_1.2.0-3.dsc
 c5ff50c85f01709d2c77bdccdcd05077 3040 otherosfs optional bchunk_1.2.0-3.diff.gz
 22cf3007e01c6477599c89ef838d97b9 13468 otherosfs optional 
bchunk_1.2.0-3_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQFFAWZV+C5cwEsrK54RAo6XAJ9j45YrgMJOiSpmtAOMqUwuzmx/FQCfWy/8
Ap8iVZSKbEZqYu1wOZ3mXKk=
=EQsR
-----END PGP SIGNATURE-----


--- End Message ---

Reply via email to