diff options
author | Alexander Barkov <bar@mariadb.com> | 2019-12-16 13:08:17 +0400 |
---|---|---|
committer | Alexander Barkov <bar@mariadb.com> | 2019-12-16 13:08:17 +0400 |
commit | 3d98892232359b7262cf82d00bc8f94b1511f26b (patch) | |
tree | b48c157fde886efc051ada08296eb6cb4f4eedc2 | |
parent | a3a8360d577b44131577c5d40336c632d35487ae (diff) | |
parent | fc860d3fa3bc854fbe6aab9179d7a4aaf6eb9edf (diff) | |
download | mariadb-git-3d98892232359b7262cf82d00bc8f94b1511f26b.tar.gz |
Merge remote-tracking branch 'origin/5.5' into 10.1
-rw-r--r-- | mysql-test/r/type_int.result | 13 | ||||
-rw-r--r-- | mysql-test/t/type_int.test | 12 | ||||
-rw-r--r-- | storage/tokudb/CMakeLists.txt | 6 | ||||
-rw-r--r-- | strings/ctype-simple.c | 3 | ||||
-rw-r--r-- | strings/my_strtoll10.c | 17 |
5 files changed, 45 insertions, 6 deletions
diff --git a/mysql-test/r/type_int.result b/mysql-test/r/type_int.result index 1364f58169e..ad2b9399464 100644 --- a/mysql-test/r/type_int.result +++ b/mysql-test/r/type_int.result @@ -20,6 +20,19 @@ COALESCE(@a) 1 DROP TABLE t1; # +# MDEV-21065 UNIQUE constraint causes a query with string comparison to omit a row in the result set +# +CREATE TABLE t1 (c0 INT UNIQUE); +INSERT INTO t1 VALUES (NULL), (NULL), (NULL), (NULL), (1), (0); +SELECT * FROM t1 WHERE c0 < '\n2'; +c0 +0 +1 +DROP TABLE t1; +SELECT CAST('\n2' AS INT); +CAST('\n2' AS INT) +2 +# # End of 5.5 tests # # diff --git a/mysql-test/t/type_int.test b/mysql-test/t/type_int.test index 8eba40c7e32..90b3945c4bf 100644 --- a/mysql-test/t/type_int.test +++ b/mysql-test/t/type_int.test @@ -15,6 +15,18 @@ SELECT COALESCE(@a) FROM t1 ORDER BY STRCMP(STDDEV_SAMP(a), 'bar'); DROP TABLE t1; --echo # +--echo # MDEV-21065 UNIQUE constraint causes a query with string comparison to omit a row in the result set +--echo # + +CREATE TABLE t1 (c0 INT UNIQUE); +INSERT INTO t1 VALUES (NULL), (NULL), (NULL), (NULL), (1), (0); +SELECT * FROM t1 WHERE c0 < '\n2'; +DROP TABLE t1; + +SELECT CAST('\n2' AS INT); + + +--echo # --echo # End of 5.5 tests --echo # diff --git a/storage/tokudb/CMakeLists.txt b/storage/tokudb/CMakeLists.txt index 618d257be29..7313a4dcce4 100644 --- a/storage/tokudb/CMakeLists.txt +++ b/storage/tokudb/CMakeLists.txt @@ -49,6 +49,12 @@ MY_CHECK_AND_SET_COMPILER_FLAG("-Wno-vla" DEBUG) MY_CHECK_AND_SET_COMPILER_FLAG("-Wno-implicit-fallthrough") MY_CHECK_AND_SET_COMPILER_FLAG("-Wno-cpp" DEBUG) +CHECK_C_COMPILER_FLAG("-Wno-address-of-packed-member" HAVE_NO_ADDRESS_PACKED) +IF(HAVE_NO_ADDRESS_PACKED) + SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-address-of-packed-member") + SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-address-of-packed-member") +ENDIF() + ############################################ SET(TOKUDB_DEB_FILES "usr/lib/mysql/plugin/ha_tokudb.so\netc/mysql/conf.d/tokudb.cnf\nusr/bin/tokuftdump" PARENT_SCOPE) MARK_AS_ADVANCED(BUILDNAME) diff --git a/strings/ctype-simple.c b/strings/ctype-simple.c index 5cbf5da7990..61d01cc6aef 100644 --- a/strings/ctype-simple.c +++ b/strings/ctype-simple.c @@ -1494,7 +1494,8 @@ my_strntoull10rnd_8bit(CHARSET_INFO *cs __attribute__((unused)), int shift= 0, digits= 0, negative, addon; /* Skip leading spaces and tabs */ - for ( ; str < end && (*str == ' ' || *str == '\t') ; str++); + for ( ; str < end && my_isspace(&my_charset_latin1, *str) ; ) + str++; if (str >= end) goto ret_edom; diff --git a/strings/my_strtoll10.c b/strings/my_strtoll10.c index ea556cea127..c6ba5666dde 100644 --- a/strings/my_strtoll10.c +++ b/strings/my_strtoll10.c @@ -98,18 +98,25 @@ longlong my_strtoll10(const char *nptr, char **endptr, int *error) if (endptr) { end= *endptr; - while (s != end && (*s == ' ' || *s == '\t')) + /* Skip leading spaces */ + for ( ; s < end && my_isspace(&my_charset_latin1, *s) ; ) s++; + if (s == end) goto no_conv; } else { endptr= &dummy; /* Easier end test */ - while (*s == ' ' || *s == '\t') - s++; - if (!*s) - goto no_conv; + /* Skip leading spaces */ + for ( ; ; s++) + { + if (!*s) + goto no_conv; + if (!my_isspace(&my_charset_latin1, *s)) + break; + } + /* This number must be big to guard against a lot of pre-zeros */ end= s+65535; /* Can't be longer than this */ } |