Re: [PATCH] util: Fix crash when removing entries during hash iteration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/12/2011 10:54 AM, Jiri Denemark wrote:
> Commit 9677cd33eea4c65d78ba463b46b8b45ed2da1709 made it possible to
> remove current entry when iterating through all hash entries. However,
> it didn't properly handle a special case of removing first entry
> assigned to a given key which contains several entries in its collision
> list.
> ---
> This is an alternate less invasive fix to "util: Simplify hash implementation".
> I prefer the more invasive fix for upstream but I'm sending this alternative
> here for completeness.

For backporting purposes, would it make sense to apply this patch first,
then apply the other one which undoes this patch as part of its
simplifications?  Then, anyone trying to pick and choose patches to
backport can feel justified going with the smaller patch in isolation.

As for this patch itself, ACK, but whether to apply it depends on the
answer to the bigger question above.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]