diff options
author | Tor Didriksen <tor.didriksen@oracle.com> | 2012-04-12 15:04:22 +0200 |
---|---|---|
committer | Tor Didriksen <tor.didriksen@oracle.com> | 2012-04-12 15:04:22 +0200 |
commit | 83d455be90a06e8fc1293a611061bd9529ed8536 (patch) | |
tree | 3a272d304a40e4f35352c9eff016ee04b0d5cc08 /sql/spatial.cc | |
parent | a84ca722247a2aa118c42d8c55d80196660d6795 (diff) | |
download | mariadb-git-83d455be90a06e8fc1293a611061bd9529ed8536.tar.gz |
Bug#13871079 RQG_MYISAM_DML_ALTER_VALGRIND FAILS ON VALGRIND PN PB2
The class Copy_field contains a String tmp,
which may allocate memory on the heap.
That means that all instances of Copy_field
must be properly destroyed. Alas they are not.
Solution: don't use Copy_field::tmp for copying
from_field => tmp => to_field
in do_field_string()
sql/field.cc:
In Field_set::val_str
return empty string (of appropriate character set) for an empty set.
sql/field.h:
New private member in Field_enum: empty_set_string.
sql/field_conv.cc:
In do_field_string, use an auto variable for copying
from_field => tmp => to_field
rather than copy->tmp.
Diffstat (limited to 'sql/spatial.cc')
0 files changed, 0 insertions, 0 deletions