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