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.