Marcus Meissner : wininet: Avoid one-byte overflows (Coverity).

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 038e3d3773d44738320f1381abdb88454ff92cdd
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=038e3d3773d44738320f1381ab...

Author: Marcus Meissner 
Date:   Thu May 30 20:27:10 2013 +0200

wininet: Avoid one-byte overflows (Coverity).

---

dlls/wininet/tests/http.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/dlls/wininet/tests/http.c b/dlls/wininet/tests/http.c
index b42e732..f4d0b43 100644
--- a/dlls/wininet/tests/http.c
+++ b/dlls/wininet/tests/http.c
@@ -2835,13 +2835,13 @@ static void test_cache_read_gzipped(int port)
     buf[size] = 0;
     ok(!strncmp(buf, content, size), "incorrect page content: %s\n", buf);

-    size = sizeof(buf);
+    size = sizeof(buf)-1;
     ret = HttpQueryInfoA(req, HTTP_QUERY_CONTENT_TYPE, buf, &size, 0);
     ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_TYPE) failed: %d\n", GetLastError());
     buf[size] = 0;
     ok(!strncmp(text_html, buf, size), "buf = %s\n", buf);

-    size = sizeof(buf);
+    size = sizeof(buf)-1;
     ret = HttpQueryInfoA(req, HTTP_QUERY_RAW_HEADERS_CRLF, buf, &size, 0);
     ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_TYPE) failed: %d\n", GetLastError());
     buf[size] = 0;
@@ -2858,7 +2858,7 @@ static void test_cache_read_gzipped(int port)
     ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0);
     ok(ret, "HttpSendRequest failed with error %u\n", GetLastError());
     size = 0;
-    while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read)
+    while(InternetReadFile(req, buf+size, sizeof(buf)-1-size, &read) && read)
         size += read;
     todo_wine ok(size == 10, "read %d bytes of data\n", size);
     buf[size] = 0;
@@ -2870,7 +2870,7 @@ static void test_cache_read_gzipped(int port)
             "HttpQueryInfo(HTTP_QUERY_CONTENT_ENCODING) returned %d, %d\n",
             ret, GetLastError());

-    size = sizeof(buf);
+    size = sizeof(buf)-1;
     ret = HttpQueryInfoA(req, HTTP_QUERY_CONTENT_TYPE, buf, &size, 0);
     todo_wine ok(ret, "HttpQueryInfo(HTTP_QUERY_CONTENT_TYPE) failed: %d\n", GetLastError());
     buf[size] = 0;
@@ -2888,7 +2888,7 @@ static void test_cache_read_gzipped(int port)
     ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0);
     ok(ret, "HttpSendRequest failed with error %u\n", GetLastError());
     size = 0;
-    while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read)
+    while(InternetReadFile(req, buf+size, sizeof(buf)-1-size, &read) && read)
         size += read;
     ok(size == 31, "read %d bytes of data\n", size);
     InternetCloseHandle(req);
@@ -2903,7 +2903,7 @@ static void test_cache_read_gzipped(int port)
     ret = HttpSendRequest(req, "Accept-Encoding: gzip", -1, NULL, 0);
     ok(ret, "HttpSendRequest failed with error %u\n", GetLastError());
     size = 0;
-    while(InternetReadFile(req, buf+size, sizeof(buf)-size, &read) && read)
+    while(InternetReadFile(req, buf+size, sizeof(buf)-1-size, &read) && read)
         size += read;
     todo_wine ok(size == 31, "read %d bytes of data\n", size);