Subject: CVS commit: pkgsrc
To: None <pkgsrc-changes@netbsd.org>
From: Alistair G. Crooks <agc@netbsd.org>
List: pkgsrc-changes
Date: 02/22/2001 13:34:59
Module Name: pkgsrc
Committed By: agc
Date: Thu Feb 22 11:34:59 UTC 2001
Modified Files:
pkgsrc/emulators/hercules/files: patch-sum
Added Files:
pkgsrc/emulators/hercules/patches: patch-af
Log Message:
Add one more patch, correcting sign extension in two places:
> From Wofgang Solfrank (ws@tools.de)
> What I discovered was a problem in ckddasd.c, where there are two lseeks
> with an offset of -(CKDDASD_RECHDR_SIZE). Now CKDDASD_RECHDR_SIZE is
> defined (in hercules.h) as sizeof(CKDDASD_RECHDR), which happens to be
> an unsigned int. Since the offset in the lseek (and in the ckd_lseek
> that is used as an interface routine to it) is of type off_t, i.e. 64-bit,
> the unsigned is zero-extended to 64 bits, and thus the sign is lost.
To generate a diff of this commit:
cvs rdiff -r1.3 -r1.4 pkgsrc/emulators/hercules/files/patch-sum
cvs rdiff -r0 -r1.1 pkgsrc/emulators/hercules/patches/patch-af
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.