Merge lp:~sergei.glushchenko/percona-server/5.6-BT40550-ps-bug1166938 into lp:percona-server/5.6

Proposed by Sergei Glushchenko
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 589
Proposed branch: lp:~sergei.glushchenko/percona-server/5.6-BT40550-ps-bug1166938
Merge into: lp:percona-server/5.6
Diff against target: 18 lines (+7/-2)
1 file modified
plugin/percona-pam-for-mysql/src/auth_pam_compat.c (+7/-2)
To merge this branch: bzr merge lp:~sergei.glushchenko/percona-server/5.6-BT40550-ps-bug1166938
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Registry Administrators Pending
Review via email: mp+219366@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

Same comment as for the percona-pam-for-mysql review. If only the comment will be changed, no need for new Jenkins.

review: Needs Fixing
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

changed comment

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 'plugin/percona-pam-for-mysql/src/auth_pam_compat.c'
--- plugin/percona-pam-for-mysql/src/auth_pam_compat.c 2013-12-23 22:05:57 +0000
+++ plugin/percona-pam-for-mysql/src/auth_pam_compat.c 2014-05-14 13:43:57 +0000
@@ -95,8 +95,13 @@
95 strncpy(resp->resp, (char *)pkt, pkt_len);95 strncpy(resp->resp, (char *)pkt, pkt_len);
96 resp->resp[pkt_len]= '\0';96 resp->resp[pkt_len]= '\0';
9797
98 /* we could only guess whether password was used or not */98 /**
99 data->info->password_used= PASSWORD_USED_NO_MENTION;99 we could only guess whether password was used or not
100 normally we would set PASSWORD_USED_NO_MENTION but
101 because of http://bugs.mysql.com/bug.php?id=72536
102 we set PASSWORD_USED_YES.
103 */
104 data->info->password_used= PASSWORD_USED_YES;
100 ++(*num_talks);105 ++(*num_talks);
101 }106 }
102107

Subscribers

People subscribed via source and target branches