]> granicus.if.org Git - curl/commit
formadd: wrong pointer for file name when CURLFORM_BUFFERPTR used
authorDaniel Stenberg <daniel@haxx.se>
Sun, 4 Aug 2013 21:27:27 +0000 (23:27 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Sun, 4 Aug 2013 21:32:36 +0000 (23:32 +0200)
commit0ddc678927eaa127efc457535858c19e791a5339
treecc8832abd8f2165e6312b3d9657b7c776f8d98af
parent51f0b798fa572496c56db62dc3970e4ea0b2760c
formadd: wrong pointer for file name when CURLFORM_BUFFERPTR used

The internal function that's used to detect known file extensions for
the default Content-Type got the the wrong pointer passed in when
CURLFORM_BUFFER + CURLFORM_BUFFERPTR were used. This had the effect that
strlen() would be used which could lead to an out-of-bounds read (and
thus segfault). In most cases it would only lead to it not finding or
using the correct default content-type.

It also showed that test 554 and test 587 were testing for the
previous/wrong behavior and now they're updated as well.

Bug: http://curl.haxx.se/bug/view.cgi?id=1262
Reported-by: Konstantin Isakov
lib/formdata.c
tests/data/test554
tests/data/test587