Jeff King wrote: > The prior commit converted many sites to check the return > value of write_in_full() for negativity, rather than a > mismatch with the input length. This patch covers similar > cases, but where the return value is stored in an > intermediate variable. These should get the same treatment, > but they need to be reviewed more carefully since it would > be a bug if the return value is stored in an unsigned type > (which indeed, it is in one of the cases). > > Signed-off-by: Jeff King <p...@peff.net> > --- > entry.c | 5 +++-- > refs/files-backend.c | 2 +- > streaming.c | 2 +- > 3 files changed, 5 insertions(+), 4 deletions(-)
Reviewed-by: Jonathan Nieder <jrnie...@gmail.com>
