> However, within context, if you are assuming there
> may be binary data read in from the stream, then
> the entire function rad_config has problems, since
> it is assuming non-binary data in buf. Therefore,
> much evasive changes are required to address this
> issue than the proposed diff.

I may be wrong, but I don't think so. I've read the rad_config function,
and it is using split function to parse the buffer returned by fgets. If
some binary data is there, rad_config will gracefully fail, because
expected data will not be there.

So my patch is legitimate, in my humble opinion.

Best regards,

Charles Longeau