Re: CVS commit: src/dist/ntp/util

[christos%zoulas.com@localhost (Christos Zoulas)]

On Nov 14,  1:32pm, explorer%flame.org@localhost (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