Subject: Re: CVS commit: src/dist/ntp/util
To: Michael Graff <explorer@flame.org>
From: Christos Zoulas <christos@zoulas.com>
List: source-changes
Date: 11/14/2006 15:26:03
On Nov 14,  1:32pm, explorer@flame.org (Michael Graff) wrote:
-- Subject: Re: CVS commit: src/dist/ntp/util

| Christos Zoulas wrote:
| > Module Name:	src
| > Committed By:	christos
| > Date:		Tue Nov 14 19:23:47 UTC 2006
| > 
| > Modified Files:
| > 	src/dist/ntp/util: ntp-keygen.c
| > 
| > Log Message:
| > openssl changed its version number from long to int; compensate.
| > 
| > 
| > To generate a diff of this commit:
| > cvs rdiff -r1.6 -r1.7 src/dist/ntp/util/ntp-keygen.c
| > 
| > Please note that diffs are not public domain; they are subject to the
| > copyright notices on the relevant files.
| > 
| 
| Note that the fix for ntpd/ntpd.c and util/ntp-keygen.c are different. 
| ntpd.c was cast to (long) and ntp-keygen.c is (unsigned int)
| 
| At least, they should match.  I'd say since %ld is the format, it seems 
| (long) is probably more correct.

Well, it is a 32 bit quantity, and it should be cast to unsigned to prevent
sign extension... But I will follow suit and do it the wrong way.

christos