[PATCH][BZ 15362] Fix fwrite() reading beyond end of buffer in error path

Siddhesh Poyarekar siddhesh@redhat.com
Wed Apr 17 22:33:00 GMT 2013


On Tue, Apr 16, 2013 at 11:31:14PM -0500, Eric Biggers wrote:
> I have not been able to run the full test suite because an unrelated test
> (math/test-ldouble) fails even without my patch.  Also, I did not add a test

Run make check with -k and record which tests fail due to this patch
(if any).

> -      /* Try to maintain alignment: write a whole number of blocks.
> -	 dont_write is what gets left over. */
> +      /* Try to maintain alignment: write a whole number of blocks. */

There should be two spaces after the end of the sentence.

The fix looks OK otherwise.  Please confirm that the rest of the
testsuite is fine after this change.  Please wait for one more
reviewer to go through this.  Once the final version of the patch is
ready after review, I can commit it for you.

Thanks,
Siddhesh



More information about the Libc-alpha mailing list