Re: [evolution-patches] fix for the bug #314925 [calendar]
- From: Harish Krishnaswamy <kharish novell com>
- To: chen <pchenthill novell com>
- Cc: patches <evolution-patches gnome org>
- Subject: Re: [evolution-patches] fix for the bug #314925 [calendar]
- Date: Sat, 01 Oct 2005 20:06:20 +0530
Looks fine. Please commit to both gnome-2-12 and HEAD.
Thanks,
Harish
On Sat, 2005-10-01 at 15:46 +0530, chen wrote:
> Have attached modified patch to fix the issue.
>
> thanks, Chenthill,
> .
> On Sat, 2005-10-01 at 13:52 +0530, Harish Krishnaswamy wrote:
> > On Wed, 2005-09-28 at 16:13 +0530, chen wrote:
> >
> > > * libecal/e-cal-component.c:
> > > + (set_attendee_list): Free the old attendee list after
> > > + adding the new attendees.
> > This is a wrong fix. The API expects you to set a new list of attendees.
> > The memory corruption happens because the caller attempts to modify the
> > existing list and send it back through the call.
> > [This case justifies, however, the need for a client API to modify
> > properties/parameters to save them overheads].
> >
> > -Harish
> >
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]