Merge lp:~sergei.glushchenko/percona-pam-for-mysql/bug1166938 into lp:percona-pam-for-mysql

Proposed by Sergei Glushchenko
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: 36
Merged at revision: 36
Proposed branch: lp:~sergei.glushchenko/percona-pam-for-mysql/bug1166938
Merge into: lp:percona-pam-for-mysql
Diff against target: 18 lines (+7/-2)
1 file modified
src/auth_pam_compat.c (+7/-2)
To merge this branch: bzr merge lp:~sergei.glushchenko/percona-pam-for-mysql/bug1166938
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Registry Administrators Pending
Review via email: mp+219369@code.launchpad.net

Description of the change

workaround mysql workbench bug http://bugs.mysql.com/bug.php?id=72536

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

The comment on the line above now goes out of sync with the code. Say instead that we'd normally pass PASSWORD_USED_NO_MENTION here, but we are working around http://bugs.mysql.com/bug.php?id=72536

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
1=== modified file 'src/auth_pam_compat.c'
2--- src/auth_pam_compat.c 2013-08-21 07:51:36 +0000
3+++ src/auth_pam_compat.c 2014-05-14 13:32:17 +0000
4@@ -95,8 +95,13 @@
5 strncpy(resp->resp, (char *)pkt, pkt_len);
6 resp->resp[pkt_len]= '\0';
7
8- /* we could only guess whether password was used or not */
9- data->info->password_used= PASSWORD_USED_NO_MENTION;
10+ /**
11+ we could only guess whether password was used or not
12+ normally we would set PASSWORD_USED_NO_MENTION but
13+ because of http://bugs.mysql.com/bug.php?id=72536
14+ we set PASSWORD_USED_YES.
15+ */
16+ data->info->password_used= PASSWORD_USED_YES;
17 ++(*num_talks);
18 }
19

Subscribers

People subscribed via source and target branches