[bug-sourceinstall] Re: invalid write

[Brian Gough]

At Sun, 06 Dec 2009 01:01:04 +0100,
Claudio Fontana wrote:
> Brian Gough wrote:
> > When _update_package(&_srcinst_state.packages, e) is called and takes
> > the branch with _remove_package_list() it causes e to be freed.  This
> > causes a memory error for info->onclose = ... because info is actually
> > set to e->info earlier in srcinst_close_package().  I'm not sure what
> > the best fix is for that -- maybe just to return after
> > _update_package()?
> 
> I think this should fix it. Can you check if it makes sense to you?

Yes, that makes sense and fixes the problem.

-- 
Brian Gough