summaryrefslogtreecommitdiffstats
path: root/contrib/diff/lib/prepargs.c
diff options
context:
space:
mode:
authortruckman <truckman@FreeBSD.org>2016-06-15 06:33:40 +0000
committertruckman <truckman@FreeBSD.org>2016-06-15 06:33:40 +0000
commitc77fcd4fd3a5d4b708c65feab05797b8074c6cb6 (patch)
tree6c4410436d23509fa819ad598753f33d71cdf6ca /contrib/diff/lib/prepargs.c
parentc2353f9c1e6539602827e9107e7f972a99f5aae3 (diff)
downloadFreeBSD-src-c77fcd4fd3a5d4b708c65feab05797b8074c6cb6.zip
FreeBSD-src-c77fcd4fd3a5d4b708c65feab05797b8074c6cb6.tar.gz
MFC r301582
Explicitly NUL terminate the buffer filled by fread(). The fix in r300649 was not sufficient to convince Coverity that the buffer was NUL terminated, even with the buffer pre-zeroed. Swap the size and nmemb arguments to fread() so that a valid lenght is returned, which we can use to terminate the string in the buffer at the correct location. This should also quiet the complaint about the return value of fread() not being checked. Reported by: Coverity CID: 1019054, 1009614 Secur3ty: Sponsore dby:
Diffstat (limited to 'contrib/diff/lib/prepargs.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud