Merge lp:~sergei.glushchenko/percona-server/5.5-ps-bug1334719 into lp:percona-server/5.5

Proposed by Sergei Glushchenko
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 700
Proposed branch: lp:~sergei.glushchenko/percona-server/5.5-ps-bug1334719
Merge into: lp:percona-server/5.5
Diff against target: 20 lines (+2/-1)
1 file modified
sql/mysqld.cc (+2/-1)
To merge this branch: bzr merge lp:~sergei.glushchenko/percona-server/5.5-ps-bug1334719
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Review via email: mp+232900@code.launchpad.net

Description of the change

    Bug 1334719: Valgrind: memory leak in MTR percona_secure_file_priv_noarg

    Release memory allocated for opt_secure_file_priv in my_getopt.c
    with my_strdup.

    Tested locally with ./mtr percona_secure_file_priv_noarg secure_file_priv --valgrind

http://jenkins.percona.com/view/PS%205.5/job/percona-server-5.5-param/1044/

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/mysqld.cc'
--- sql/mysqld.cc 2014-08-22 09:58:37 +0000
+++ sql/mysqld.cc 2014-09-01 14:53:56 +0000
@@ -432,7 +432,7 @@
432my_bool relay_log_recovery;432my_bool relay_log_recovery;
433my_bool opt_sync_frm, opt_allow_suspicious_udfs;433my_bool opt_sync_frm, opt_allow_suspicious_udfs;
434my_bool opt_secure_auth= 0;434my_bool opt_secure_auth= 0;
435char* opt_secure_file_priv;435char* opt_secure_file_priv= NULL;
436my_bool opt_secure_file_priv_noarg= FALSE;436my_bool opt_secure_file_priv_noarg= FALSE;
437my_bool opt_log_slow_admin_statements= 0;437my_bool opt_log_slow_admin_statements= 0;
438my_bool opt_log_slow_slave_statements= 0;438my_bool opt_log_slow_slave_statements= 0;
@@ -7521,6 +7521,7 @@
7521 case OPT_SECURE_FILE_PRIV:7521 case OPT_SECURE_FILE_PRIV:
7522 if (argument == NULL)7522 if (argument == NULL)
7523 {7523 {
7524 my_free(opt_secure_file_priv);
7524 opt_secure_file_priv_noarg= TRUE;7525 opt_secure_file_priv_noarg= TRUE;
7525 opt_secure_file_priv= my_strdup("ON", MYF(0));7526 opt_secure_file_priv= my_strdup("ON", MYF(0));
7526 }7527 }

Subscribers

People subscribed via source and target branches