DISALLOW_REUSE line not correctly maintained in pam module about google-authenticator HOT 5 CLOSED

Comment #1 originally posted by valient on 2011-01-18T08:36:04.000Z:

Looks like the following change to pam_google_authenticator.c fixes it:

@@ -439,6 +440,7 @@
// remove it from the file.
if (blocked - tm > 3 || tm - blocked > 3) {
endptr += strspn(endptr, " \t");

•  ptr += strspn(ptr, " \t");
  

  memmove(ptr, endptr, strlen(endptr) + 1);
  memset(strrchr(ptr, '\000'), 0, endptr - ptr + 1);
  } else {

from google-authenticator.

Comment #2 originally posted by vittgam on 2011-02-27T16:29:39.000Z:

I've fixed this issue at https://github.com/VittGam/google-authenticator following comment 1 suggestion.

from google-authenticator.

Comment #3 originally posted by [email protected] on 2011-03-09T20:35:17.000Z:

<empty>

from google-authenticator.

Comment #4 originally posted by [email protected] on 2011-03-09T20:35:51.000Z:

Issue 28 has been merged into this issue.

from google-authenticator.

Comment #5 originally posted by [email protected] on 2011-03-10T08:47:21.000Z:

This is now tested for in the unittest and fixed in the code. Thank you for the very detailed bug report and for even attaching a patch. I ended up fixing things a little differently, but the patch helped in explaining the bug.

from google-authenticator.