Merge lp:~laurynas-biveinis/percona-server/bug1067103-5.5 into lp:percona-server/5.5

Proposed by Laurynas Biveinis
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 708
Proposed branch: lp:~laurynas-biveinis/percona-server/bug1067103-5.5
Merge into: lp:percona-server/5.5
Diff against target: 12 lines (+1/-1)
1 file modified
sql/sys_vars.cc (+1/-1)
To merge this branch: bzr merge lp:~laurynas-biveinis/percona-server/bug1067103-5.5
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Review via email: mp+238384@code.launchpad.net

Description of the change

Fix bug 1067103 (Out-of-bound reads due to incorrect definition of
log_warnings_suppress_name) by terminating the
log_warnings_suppress_name array by a NULL pointer.

Tested locally by compiling and running percona_log_warnings_suppress test.

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sql/sys_vars.cc'
--- sql/sys_vars.cc 2014-09-24 09:00:06 +0000
+++ sql/sys_vars.cc 2014-10-15 05:02:30 +0000
@@ -1645,7 +1645,7 @@
1645 READ_ONLY GLOBAL_VAR(mysqld_port), CMD_LINE(REQUIRED_ARG, 'P'),1645 READ_ONLY GLOBAL_VAR(mysqld_port), CMD_LINE(REQUIRED_ARG, 'P'),
1646 VALID_RANGE(0, UINT_MAX32), DEFAULT(0), BLOCK_SIZE(1));1646 VALID_RANGE(0, UINT_MAX32), DEFAULT(0), BLOCK_SIZE(1));
16471647
1648const char *log_warnings_suppress_name[]= { "1592" };1648static const char *log_warnings_suppress_name[]= { "1592", NullS };
1649static Sys_var_set Sys_log_warnings_suppress(1649static Sys_var_set Sys_log_warnings_suppress(
1650 "log_warnings_suppress",1650 "log_warnings_suppress",
1651 "disable logging of enumerated warnings: "1651 "disable logging of enumerated warnings: "

Subscribers

People subscribed via source and target branches