[parted-devel] release plan of parted 1.8.9
Mike Hwang
Mike.Hwang at iqstor.com.cn
Wed Jul 30 10:22:22 UTC 2008
I run command mkpart.
The patch works.
One question, what does "make check" do?
On Wed, 2008-07-30 at 11:11 +0200, Jim Meyering wrote:
> Mike Hwang <Mike.Hwang at iqstor.com.cn> wrote:
>
> > ==13891== 80 bytes in 4 blocks are definitely lost in loss record 14 of
> > 35
> > ==13891== at 0x40214E5: malloc (vg_replace_malloc.c:149)
> > ==13891== by 0x80D6884: ped_malloc (libparted.c:234)
> > ==13891== by 0x81165CB: _disk_sync_part_table (linux.c:2216)
> > ==13891== by 0x8116EEC: linux_disk_commit (linux.c:2514)
> > ==13891== by 0x80D8B23: ped_disk_commit_to_os (disk.c:445)
> >
> > int* rets = ped_malloc(sizeof(int) * last); (linux.c:2216)
>
> Thanks.
> For reference, what parted command were you running?
> If it's not yet done as part of "make check", I'll add it.
>
> Please confirm that this patch fixes it.
>
> >From 793f272c8ef1830aadb57159957235a3f6e33cfa Mon Sep 17 00:00:00 2001
> From: Jim Meyering <meyering at redhat.com>
> Date: Wed, 30 Jul 2008 11:10:56 +0200
> Subject: [PATCH] linux.c: plug a leak
>
> * libparted/arch/linux.c (_disk_sync_part_table): Free local, "rets".
> Reported by Mike Hwang <Mike.Hwang at iqstor.com.cn>.
> ---
> libparted/arch/linux.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/libparted/arch/linux.c b/libparted/arch/linux.c
> index 19ddae5..75e709b 100644
> --- a/libparted/arch/linux.c
> +++ b/libparted/arch/linux.c
> @@ -2240,6 +2240,7 @@ _disk_sync_part_table (PedDisk* disk)
> }
> }
>
> + free (rets);
> free (errnums);
> return ret;
> }
> --
> 1.6.0.rc1.2.gc4577
>
--
Mike Hwang
http://www.iqstor.com
Phone: 62974477-612
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.alioth.debian.org/pipermail/parted-devel/attachments/20080730/5ef306fc/attachment.htm
More information about the parted-devel
mailing list