RT RT/krbdev.mit.edu: Ticket #6029 kadmind leaks error strings on failures Signed in as guest.
[Logout]

[Home] [Search] [Configuration]

[Display] [History] [Basics] [Dates] [People] [Links] [Jumbo]

 
 

 The Basics  
Id
6029
Status
resolved
Worked
0 min
Priority
50/50
Queue
krb5
 

 Keyword Selections  
Component
  • krb5-admin
Version_reported
  • 1.6.3
Version_Fixed
  • 1.7
Target_Version
Tags
 

 Relationships  
Depends on:
Depended on by:
Parents:
Children:

Refers to:
Referred to by:
  • 6202: (ghudson) kadmind leaks extended error strings [resolved]
 
 Dates  
Created: Mon Jul 14 07:05:25 2008
Starts: Not set
Started: Thu Oct 23 19:53:11 2008
Last Contact: Mon Nov 3 13:34:16 2008
Due: Not set
Updated: Fri Jan 30 23:08:19 2009 by tlyu
 

 People  
Owner
 ghudson
Requestors
 Mark.Phalan@Sun.Com
Cc
 Mark.Phalan@Sun.Com
AdminCc
 
 

 More about Mark.Phalan@Sun.Com  
Comments about this user:
No comment entered about this user
This user's 25 highest priority tickets:
 

History   Display mode: [Brief headers] [Full headers]
      Mon Jul 14 07:05:26 2008  Mark.Phalan@Sun.Com - Ticket created    
     
CC: Mark.Phalan@Sun.Com

 

     

On pretty much every failed kadmin "action" memory is leaked from
krb5_get_error_message(). The attached patch krb5_free_error_message()'s
the error message before returning.

Download (untitled) 175b
     
 
Download krb5-1.6.3_kadmin_memleak_fixes.patch 19.2k
      Mon Jul 14 16:46:56 2008  tlyu - Subject changed from to kadmind leaks error strings on failures    
      Mon Jul 14 16:46:56 2008  tlyu - Component krb5-admin added    
      Mon Jul 14 16:46:57 2008  tlyu - Version_reported 1.6.3 added    
      Thu Oct 23 19:53:11 2008  tlyu - FinalPriority changed from 0 to 50    
      Thu Oct 23 19:53:11 2008  tlyu - Priority changed from 0 to 50    
      Thu Oct 23 19:53:11 2008  tlyu - Status changed from new to open    
      Thu Oct 23 19:53:11 2008  tlyu - Given to ghudson    
      Mon Nov  3 13:34:16 2008  ghudson - Status changed from open to review    
      Mon Nov  3 13:34:16 2008  ghudson - Correspondence added    
     
Adapted and applied.

I accidentally tagged the commit for #6202, which is a related issue
(Sun patch for some of the same leaks).  r20956 is the relevant commit.


Download (untitled) 162b
      Fri Jan 23 17:22:26 2009  tlyu - Status changed from review to resolved    
      Fri Jan 30 23:08:19 2009  tlyu - Version_Fixed 1.7 added