mirror/mariadb
1
0
Fork 0
mirror of https://github.com/MariaDB/server.git synced 2025-01-16 12:02:42 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

String::append_for_single_quote() should signal OOM condition,

Browse source 
just like other String::append() methods do
This commit is contained in:
Sergei Golubchik 2013-04-19 20:35:32 +02:00
parent 4b169cd7c1
commit faf4d99d22
2 changed files with 11 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

31
sql/sql_string.cc
View file

@ -1079,7 +1079,8 @@ outp:
characters as necessary.
Does not add the enclosing quotes, this is left up to caller.
*/
void String::append_for_single_quote(const char *st, uint len)
#define APPEND(X) if (append(X)) return 1; else break
bool String::append_for_single_quote(const char *st, uint len)
{
const char *end= st+len;
for (; st < end; st++)
@ -1087,28 +1088,16 @@ void String::append_for_single_quote(const char *st, uint len)
uchar c= *st;
switch (c)
{
case '\\':
append(STRING_WITH_LEN("\\\\"));
break;
case '\0':
append(STRING_WITH_LEN("\\0"));
break;
case '\'':
append(STRING_WITH_LEN("\\'"));
break;
case '\n':
append(STRING_WITH_LEN("\\n"));
break;
case '\r':
append(STRING_WITH_LEN("\\r"));
break;
case '\032': // Ctrl-Z
append(STRING_WITH_LEN("\\Z"));
break;
default:
append(c);
case '\\': APPEND(STRING_WITH_LEN("\\\\"));
case '\0': APPEND(STRING_WITH_LEN("\\0"));
case '\'': APPEND(STRING_WITH_LEN("\\'"));
case '\n': APPEND(STRING_WITH_LEN("\\n"));
case '\r': APPEND(STRING_WITH_LEN("\\r"));
case '\032': APPEND(STRING_WITH_LEN("\\Z"));
default: APPEND(c);
}
}
return 0;
}
void String::print(String *str)

2
sql/sql_string.h
View file

@ -476,7 +476,7 @@ public:
return FALSE;
}
void print(String *print);
void append_for_single_quote(const char *st, uint len);
bool append_for_single_quote(const char *st, uint len);
/* Swap two string objects. Efficient way to exchange data without memcpy. */
void swap(String &s);