Menu

#3 Memory leak problem

closed-fixed
None
5
2002-08-26
2000-11-23
Vincent Tam
No

When using pam_mysql.so with Debian 2.2's CyrusIMAPd v1.5.19 pwcheck daemon,
we found that every authentication via pam_mysql.so add up 24k to the pwcheck's RSS.
Since the pwcheck daemon won't quit after an authentication, we then have a record of 54M RSS
in 3 days uptime at our server!

Suggested solution:
insert:
mysql_free_result(result);

before line 214:
return PAM_AUTH_ERR;

and before line 229:
return retvalue;

This works for us.

Discussion

  • Vincent Tam

    Vincent Tam - 2000-11-23

    Forgot to mention, this problem occur in latest v0.4.7 release

     
  • Julián Muńoz

    Julián Muńoz - 2002-01-22

    Logged In: YES
    user_id=77756

    Looking all the bugs, this one seems the more problematic.

    Do you know if it this has been corrected in the cvs ??
    The last important correction I see is the one from Kyle:

    pam_mysql.c revision 1.10
    fixing this:
    [patch #103719] fix for bug #131467 (segfault)
    Fixes buffer overflow in parseArgs()

    Anyone has got the same problem described here ?
    Is it happening in the debian package 0.4.7 ? (This bug is
    not registered in debian.org)

    Vincent, How good is you suggested solution, after more than
    one year ?

    Thank you in advance,

    Julin

     
  • James O'Kane

    James O'Kane - 2002-08-26

    Logged In: YES
    user_id=7767

    I just added the suggested changes to cvs. However, I dont'
    have a debian machine to test with. It compiles cleanly. :)
    If possible, please test.

     
  • James O'Kane

    James O'Kane - 2002-08-26
    • assigned_to: nobody --> jo2y
    • status: open --> open-fixed
     
  • James O'Kane

    James O'Kane - 2002-08-26
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.