From: drh <drh@noemail.net>
Date: Fri, 16 Jan 2004 13:58:18 +0000 (+0000)
Subject: Fix a buffer-overflow problem in the randStr function (used only for testing). (CVS... 
X-Git-Tag: version-3.6.10~4886
X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=1dba72794659c85f3eef0139da5f349f2f683259;p=thirdparty%2Fsqlite.git

Fix a buffer-overflow problem in the randStr function (used only for testing). (CVS 1182)

FossilOrigin-Name: 42c79edc2e8d1051b3bca915b4b205c601b8077f
---

diff --git a/manifest b/manifest
index 8093b607a4..896ac5261a 100644
--- a/manifest
+++ b/manifest
@@ -1,5 +1,5 @@
-C More\saggressive\sretry\sschedule\sin\ssqlite_busy_timeout().\s(CVS\s1181)
-D 2004-01-15T13:29:32
+C Fix\sa\sbuffer-overflow\sproblem\sin\sthe\srandStr\sfunction\s(used\sonly\sfor\stesting).\s(CVS\s1182)
+D 2004-01-16T13:58:18
 F Makefile.in 0515ff9218ad8d5a8f6220f0494b8ef94c67013b
 F Makefile.linux-gcc b86a99c493a5bfb402d1d9178dcdc4bd4b32f906
 F README f1de682fbbd94899d50aca13d387d1b3fd3be2dd
@@ -32,7 +32,7 @@ F src/date.c bb89fdb9c89e367b9a728c58cb96e4823974a2c1
 F src/delete.c 0f81e6799c089487615d38e042a2de4d2d6192bc
 F src/encode.c 9e70ea1e4e746f23f18180949e94f1bb1c2220d3
 F src/expr.c 866a6d7aacc2825aa13056ccbea1a16f436a1ca5
-F src/func.c 62cf8fae8147c0301d1c6a4a94fe0a78f7aa5b33
+F src/func.c 564c0bbe93c290774b305c0199237b8e8bcbda53
 F src/hash.c 9b56ef3b291e25168f630d5643a4264ec011c70e
 F src/hash.h 3247573ab95b9dd90bcca0307a75d9a16da1ccc7
 F src/insert.c 01f66866f35c986eab4a57373ca689a3255ef2df
@@ -180,7 +180,7 @@ F www/speed.tcl 2f6b1155b99d39adb185f900456d1d592c4832b3
 F www/sqlite.tcl 3c83b08cf9f18aa2d69453ff441a36c40e431604
 F www/tclsqlite.tcl b9271d44dcf147a93c98f8ecf28c927307abd6da
 F www/vdbe.tcl 9b9095d4495f37697fd1935d10e14c6015e80aa1
-P 01874d252ac44861e927dea3f5534f67e19b1fa8
-R 889c699198a4172a6f24b672e8a9f759
+P 5e85025be7aa4a03b0cfb4d0f28a2e44653b9d3f
+R aaef1a0aff5b12b7d4f36cc6a72499f1
 U drh
-Z 4f8f302ff0837a7468fa75430d63c5d0
+Z ebbe741602d66e22ec8fa796a8084911
diff --git a/manifest.uuid b/manifest.uuid
index 3d91402ba1..42f947c657 100644
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-5e85025be7aa4a03b0cfb4d0f28a2e44653b9d3f
\ No newline at end of file
+42c79edc2e8d1051b3bca915b4b205c601b8077f
\ No newline at end of file
diff --git a/src/func.c b/src/func.c
index 643a1513bc..5b5c86613e 100644
--- a/src/func.c
+++ b/src/func.c
@@ -16,7 +16,7 @@
 ** sqliteRegisterBuildinFunctions() found at the bottom of the file.
 ** All other code has file scope.
 **
-** $Id: func.c,v 1.35 2004/01/02 13:17:49 drh Exp $
+** $Id: func.c,v 1.36 2004/01/16 13:58:18 drh Exp $
 */
 #include <ctype.h>
 #include <math.h>
@@ -358,7 +358,7 @@ static void randStr(sqlite_func *context, int argc, const char **argv){
   if( argc>=2 ){
     iMax = atoi(argv[1]);
     if( iMax<iMin ) iMax = iMin;
-    if( iMax>=sizeof(zBuf) ) iMax = sizeof(zBuf);
+    if( iMax>=sizeof(zBuf) ) iMax = sizeof(zBuf)-1;
   }else{
     iMax = 50;
   }
@@ -367,6 +367,7 @@ static void randStr(sqlite_func *context, int argc, const char **argv){
     r = sqliteRandomInteger() & 0x7fffffff;
     n += r%(iMax + 1 - iMin);
   }
+  assert( n<sizeof(zBuf) );
   r = 0;
   for(i=0; i<n; i++){
     r = (r + sqliteRandomByte())% (sizeof(zSrc)-1);