From: drh <drh@noemail.net>
Date: Fri, 1 Nov 2019 10:49:15 +0000 (+0000)
Subject: Fix a potential array bounds overflow in the mkkeywordhash.c code generator.
X-Git-Tag: version-3.31.0~357
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=e7d9814fe4691bf75d5cb29ab9e7bea67e8159ef;p=thirdparty%2Fsqlite.git

Fix a potential array bounds overflow in the mkkeywordhash.c code generator.
Also add marks to omit keywords specific to generated columns when building
with -DSQLITE_OMIT_GENERATED_COLUMNS.

FossilOrigin-Name: cc6a40818387f78f89499f09e3f1c4655c7396af1cba2596c7fb2f23f3e9755f
---

diff --git a/manifest b/manifest
index aef50dc085..b70e3b01ee 100644
--- a/manifest
+++ b/manifest
@@ -1,5 +1,5 @@
-C Slightly\sfaster\skeyword\shash\stable.
-D 2019-11-01T02:30:54.247
+C Fix\sa\spotential\sarray\sbounds\soverflow\sin\sthe\smkkeywordhash.c\scode\sgenerator.\nAlso\sadd\smarks\sto\somit\skeywords\sspecific\sto\sgenerated\scolumns\swhen\sbuilding\nwith\s-DSQLITE_OMIT_GENERATED_COLUMNS.
+D 2019-11-01T10:49:15.808
 F .fossil-settings/empty-dirs dbb81e8fc0401ac46a1491ab34a7f2c7c0452f2f06b54ebb845d024ca8283ef1
 F .fossil-settings/ignore-glob 35175cdfcf539b2318cb04a9901442804be81cd677d8b889fcc9149c21f239ea
 F LICENSE.md df5091916dbb40e6e9686186587125e1b2ff51f022cc334e886c19a0e9982724
@@ -1777,7 +1777,7 @@ F tool/max-limits.c cbb635fbb37ae4d05f240bfb5b5270bb63c54439
 F tool/mkautoconfamal.sh 422fc365358a2e92876ffc62971a0ff28ed472fc8bcf9de0df921c736fdeca5e
 F tool/mkccode.tcl 86463e68ce9c15d3041610fedd285ce32a5cf7a58fc88b3202b8b76837650dbe x
 F tool/mkctimec.tcl dd183b73ae1c28249669741c250525f0407e579a70482371668fd5f130d9feb3
-F tool/mkkeywordhash.c 1d4626fecd48b32a84963b663b34f3a42ee77691a21614bdacdc71d9412d5da4
+F tool/mkkeywordhash.c 27ffc6f6e7e3ecbfc5bca1f1f11a09fc5badf6d67557a5fb2d3b069dbed90617
 F tool/mkmsvcmin.tcl cad0c7b54d7dd92bc87d59f36d4cc4f070eb2e625f14159dc2f5c4204e6a13ea
 F tool/mkopcodec.tcl d1b6362bd3aa80d5520d4d6f3765badf01f6c43c
 F tool/mkopcodeh.tcl 352a4319c0ad869eb26442bf7c3b015aa15594c21f1cce5a6420dbe999367c21
@@ -1849,7 +1849,7 @@ F vsixtest/vsixtest.tcl 6a9a6ab600c25a91a7acc6293828957a386a8a93
 F vsixtest/vsixtest.vcxproj.data 2ed517e100c66dc455b492e1a33350c1b20fbcdc
 F vsixtest/vsixtest.vcxproj.filters 37e51ffedcdb064aad6ff33b6148725226cd608e
 F vsixtest/vsixtest_TemporaryKey.pfx e5b1b036facdb453873e7084e1cae9102ccc67a0
-P 40d3282ec285d9f724f6548283f48b601510cf5284da17485723fd650a68f436
-R c292184f9e256f8778bc51749362a73d
+P f12e743e19a04ecbf7eb69b675082f2e4dda05b38cd81f6588a1ce95dfc57ada
+R a19e7b395a0485b418d66f725ce1e9ee
 U drh
-Z f600eb4b341234df9d684e7b16c76669
+Z 60eb5202ef4aac4c6596aee0e04e6735
diff --git a/manifest.uuid b/manifest.uuid
index 166b2dd41a..f0de16cbea 100644
--- a/manifest.uuid
+++ b/manifest.uuid
@@ -1 +1 @@
-f12e743e19a04ecbf7eb69b675082f2e4dda05b38cd81f6588a1ce95dfc57ada
\ No newline at end of file
+cc6a40818387f78f89499f09e3f1c4655c7396af1cba2596c7fb2f23f3e9755f
\ No newline at end of file
diff --git a/tool/mkkeywordhash.c b/tool/mkkeywordhash.c
index ef7684270a..e4393e8a5a 100644
--- a/tool/mkkeywordhash.c
+++ b/tool/mkkeywordhash.c
@@ -154,6 +154,11 @@ struct Keyword {
 #else
 #  define WINDOWFUNC 0x00100000
 #endif
+#ifdef SQLITE_OMIT_GENERATED_COLUMNS
+#  define GENCOL 0
+#else
+#  define GENCOL 0x00200000
+#endif
 
 /*
 ** These are the keywords
@@ -165,7 +170,7 @@ static Keyword aKeywordTable[] = {
   { "AFTER",            "TK_AFTER",        TRIGGER,          0      },
   { "ALL",              "TK_ALL",          ALWAYS,           0      },
   { "ALTER",            "TK_ALTER",        ALTER,            0      },
-  { "ALWAYS",           "TK_ALWAYS",       ALWAYS,           0      },
+  { "ALWAYS",           "TK_ALWAYS",       GENCOL,           0      },
   { "ANALYZE",          "TK_ANALYZE",      ANALYZE,          0      },
   { "AND",              "TK_AND",          ALWAYS,           10     },
   { "AS",               "TK_AS",           ALWAYS,           10     },
@@ -218,7 +223,7 @@ static Keyword aKeywordTable[] = {
   { "FOREIGN",          "TK_FOREIGN",      FKEY,             1      },
   { "FROM",             "TK_FROM",         ALWAYS,           10     },
   { "FULL",             "TK_JOIN_KW",      ALWAYS,           3      },
-  { "GENERATED",        "TK_GENERATED",    ALWAYS,           1      },
+  { "GENERATED",        "TK_GENERATED",    GENCOL,           1      },
   { "GLOB",             "TK_LIKE_KW",      ALWAYS,           3      },
   { "GROUP",            "TK_GROUP",        ALWAYS,           5      },
   { "GROUPS",           "TK_GROUPS",       WINDOWFUNC,       2      },
@@ -364,7 +369,9 @@ static Keyword *findById(int id){
 */
 static void reorder(int *pFrom){
   int i = *pFrom - 1;
-  int j = aKeywordTable[i].iNext;
+  int j;
+  if( i<0 ) return;
+  j = aKeywordTable[i].iNext;
   if( j==0 ) return;
   j--;
   if( aKeywordTable[i].priority >= aKeywordTable[j].priority ) return;