From jylefort@brutele.be  Wed Apr 19 23:18:52 2006
Return-Path: <jylefort@brutele.be>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 7090916A401
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 19 Apr 2006 23:18:52 +0000 (UTC)
	(envelope-from jylefort@brutele.be)
Received: from mirapoint3.brutele.be (mirapoint3.brutele.be [212.68.199.148])
	by mx1.FreeBSD.org (Postfix) with ESMTP id D47B043D45
	for <FreeBSD-gnats-submit@freebsd.org>; Wed, 19 Apr 2006 23:18:51 +0000 (GMT)
	(envelope-from jylefort@brutele.be)
Received: from host-212-68-244-243.brutele.be (host-212-68-244-243.brutele.be [212.68.244.243])
	by mirapoint3.brutele.be (MOS 3.7.3-GA)
	with ESMTP id AFL12059;
	Thu, 20 Apr 2006 01:18:49 +0200 (CEST)
Received: from jsite.lefort.net (jsite.lefort.net [192.168.1.2])
	by gateway.lefort.net (Postfix) with ESMTP id 877505528
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 20 Apr 2006 01:18:49 +0200 (CEST)
Received: by jsite.lefort.net (Postfix, from userid 1000)
	id 3E9C7C108; Thu, 20 Apr 2006 01:18:49 +0200 (CEST)
Message-Id: <20060419231849.3E9C7C108@jsite.lefort.net>
Date: Thu, 20 Apr 2006 01:18:49 +0200 (CEST)
From: Jean-Yves Lefort <jylefort@FreeBSD.org>
Reply-To: Jean-Yves Lefort <jylefort@FreeBSD.org>
To: FreeBSD-gnats-submit@freebsd.org
Subject: memory leak in libdisk's Disk_Names()
X-Send-Pr-Version: 3.113

>Number:         96077
>Category:       kern
>Synopsis:       [libdisk] memory leak in libdisk(3)'s Disk_Names()
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    antoine
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Wed Apr 19 23:20:16 GMT 2006
>Closed-Date:    Mon Mar 17 19:12:45 UTC 2008
>Last-Modified:  Mon Mar 17 19:12:45 UTC 2008
>Originator:     Jean-Yves Lefort
>Release:        FreeBSD 6.0-RELEASE i386
>Organization:
>Environment:
System: FreeBSD jsite.lefort.net 6.0-RELEASE FreeBSD 6.0-RELEASE #0: Wed Apr 19 13:51:26 CEST 2006 jylefort@jsite.lefort.net:/usr/obj/usr/src/sys/JSITE i386
>Description:
disklist is leaked away. And the manpage is lying: "you must free each
pointer, as well as the array by hand".
>How-To-Repeat:
>Fix:
>Release-Note:
>Audit-Trail:

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/96077: commit references a PR
Date: Fri, 15 Feb 2008 21:19:25 +0000 (UTC)

 antoine     2008-02-15 21:19:15 UTC
 
   FreeBSD src repository
 
   Modified files:
     lib/libdisk          disk.c 
   Log:
   - Make Disk_Names() behave as documented in libdisk(3): return an array
   of disk names, where you must free each pointer, as well as the array
   by hand. [1]
   - Destaticize "disks" in Disk_Names, it has no reasons to be static.
   
   PR:             kern/96077 [1]
   PR:             kern/114110 [1]
   MFC after:      1 month
   Approved by:    rwatson (mentor)
   
   Revision  Changes    Path
   1.128     +14 -5     src/lib/libdisk/disk.c
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: open->patched 
State-Changed-By: antoine 
State-Changed-When: Fri Feb 15 21:20:51 UTC 2008 
State-Changed-Why:  
patched in revision 1.128 of src/lib/libdisk/disk.c 


Responsible-Changed-From-To: freebsd-bugs->antoine 
Responsible-Changed-By: antoine 
Responsible-Changed-When: Fri Feb 15 21:20:51 UTC 2008 
Responsible-Changed-Why:  
patched in revision 1.128 of src/lib/libdisk/disk.c 

http://www.freebsd.org/cgi/query-pr.cgi?pr=96077 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/96077: commit references a PR
Date: Mon, 17 Mar 2008 19:05:41 +0000 (UTC)

 antoine     2008-03-17 19:05:36 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_7)
     lib/libdisk          disk.c 
   Log:
   MFC to RELENG_7
     - Make Disk_Names() behave as documented in libdisk(3): return an array
     of disk names, where you must free each pointer, as well as the array
     by hand. [1]
     - Destaticize "disks" in Disk_Names, it has no reasons to be static.
   
     PR:             kern/96077 [1]
     PR:             kern/114110 [1]
     MFC after:      1 month
     Approved by:    rwatson (mentor)
   
   Revision   Changes    Path
   1.127.2.1  +14 -5     src/lib/libdisk/disk.c
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: kern/96077: commit references a PR
Date: Mon, 17 Mar 2008 19:08:38 +0000 (UTC)

 antoine     2008-03-17 19:08:33 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_6)
     lib/libdisk          disk.c 
   Log:
   MFC to RELENG_6
     - Make Disk_Names() behave as documented in libdisk(3): return an array
     of disk names, where you must free each pointer, as well as the array
     by hand. [1]
     - Destaticize "disks" in Disk_Names, it has no reasons to be static.
   
     PR:             kern/96077 [1]
     PR:             kern/114110 [1]
     MFC after:      1 month
     Approved by:    rwatson (mentor)
   
   Revision   Changes    Path
   1.125.2.2  +14 -5     src/lib/libdisk/disk.c
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: patched->closed 
State-Changed-By: antoine 
State-Changed-When: Mon Mar 17 19:11:42 UTC 2008 
State-Changed-Why:  
Close: fixed in HEAD, RELENG_7 and RELENG_6. 
Thanks for the report. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=96077 
>Unformatted:
