[Bug 269] Many compiler warnings with gcc 4.0

Alan DeKok aland at ox.org
Wed Aug 31 22:09:49 CEST 2005


Frank Cusack <fcusack at fcusack.com> wrote:
> Forcing unsigned char would be a mistake, IMHO.  I personally think casts
> are good.  One man's ugly notation is another man's inline documentation.

  The casts hilight API incompatibilities, and potential security
issues.

  Maybe since we're doing massive re-writes anyhow, we could have:

VALUE_PAIR {
	 union {
	       char strvalue[];
	       uint8_t octets[];
	       ...	 
	 } data;
}

  And then use the "right" data member to access the data.  "uint8_t"
shouldn't be passed to "strcpy", especially if we're using "uint8_t"
as "opaque data".

  Alan DeKok.




More information about the Freeradius-Devel mailing list