Skip Menu |
 

History Show all quoted text — Show full headers
# Tue Oct 21 17:01:19 2008 lxs (Alexandra Ellwood) - Ticket created
Subject: krb5_change_set_password not freeing chpw_rep contents
Download (untitled) / with headers
text/plain 119B
krb5_change_set_password calls krb5int_sendto which places the reply buffer data in chpw_rep.
This data is not freed.
# Tue Oct 21 17:05:30 2008 lxs (Alexandra Ellwood) - Status changed from 'new' to 'review'
# Tue Oct 21 17:05:30 2008 lxs (Alexandra Ellwood) - Correspondence added
From: lxs@mit.edu
Subject: SVN Commit
Download (untitled) / with headers
text/plain 143B

krb5_change_set_password should free chpw_rep contents

Commit By: lxs



Revision: 20902
Changed Files:
U trunk/src/lib/krb5/os/changepw.c
# Tue Oct 21 17:05:40 2008 lxs (Alexandra Ellwood) - Ticket #6215: - Ticket created
Subject: krb5_change_set_password not freeing chpw_rep contents
Download (untitled) / with headers
text/plain 119B
krb5_change_set_password calls krb5int_sendto which places the reply buffer data in chpw_rep.
This data is not freed.
# Tue Oct 21 17:06:50 2008 lxs (Alexandra Ellwood) - Ticket #6215: - Ticket 6215 MergedInto ticket 6214.
# Tue Oct 21 17:07:40 2008 lxs (Alexandra Ellwood) - Given to ghudson@mit.edu (Greg Hudson)
# Tue Oct 21 17:07:40 2008 lxs (Alexandra Ellwood) - Comments added
Download (untitled) / with headers
text/plain 89B
Fixed, but passing onto Greg so he can check if the static analysis tools found this one.
# Wed Oct 22 14:12:27 2008 ghudson@mit.edu (Greg Hudson) - Status changed from 'review' to 'resolved'
# Wed Oct 22 14:12:27 2008 ghudson@mit.edu (Greg Hudson) - Correspondence added
Download (untitled) / with headers
text/plain 360B
[lxs - Tue Oct 21 17:07:40 2008]:

Show quoted text
> Fixed, but passing onto Greg so he can check if the static analysis
> tools found this one.

Doesn't look like the tool caught this. I don't know how I could find
out why; but this is a good indication that runtime leak analysis will
find different leaks than Coverity status analysis.

(Patch reviewed, resolving ticket.)
# Fri Jan 30 23:08:20 2009 tlyu (Taylor Yu) - Version_Fixed 1.7 added