diff options
author | Ray Lai <ray@cvs.openbsd.org> | 2008-03-15 21:54:10 +0000 |
---|---|---|
committer | Ray Lai <ray@cvs.openbsd.org> | 2008-03-15 21:54:10 +0000 |
commit | 50316be7bf1b3797fd1196f7cdac27a1a2fa905c (patch) | |
tree | 3c1e08fef2a7476ea3b7bc95d63d710c1a149509 /usr.sbin | |
parent | 2d2da217eeb2b589e8671b4211fdb6b8c3431b57 (diff) |
- len is size_t, but n uses len and is an int. Matching those types
should be good, plus it prevents weird things from happening if
len > INT_MAX.
- Since n is now size_t, compare it against 0 instead of >= 0.
- temp is used to store individual bytes, so use char instead
(matches fp and tp).
- millert noted that the comma operator may not guarantee order of
execution, so replace with semicolons.
Found by lint, OK millert.
Diffstat (limited to 'usr.sbin')
0 files changed, 0 insertions, 0 deletions