Re: [PATCH] Prevent crash from dlclose() of libvirt.so

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

 



On 09/01/2011 11:12 AM, Eric Blake wrote:
On 09/01/2011 10:57 AM, Daniel P. Berrange wrote:
+++ b/src/Makefile.am
@@ -1228,7 +1228,7 @@ libvirt_qemu.def: $(srcdir)/libvirt_qemu.syms
libvirt_la_SOURCES =
libvirt_la_LDFLAGS = $(VERSION_SCRIPT_FLAGS)$(LIBVIRT_SYMBOL_FILE) \
-version-info $(LIBVIRT_VERSION_INFO) \
- $(AM_LDFLAGS) \
+ $(AM_LDFLAGS) $(LIBVIRT_NODELETE) \

I would have listed this before $(AM_LDFLAGS), closer to the
$(LIBVIRT_VERSION_INFO) linker flags.


...

 > +
 > + /* If we got to hear the thread succesfully exited without
 > + * causing a SEGV !

s/hear/here/; s/succesfully/successfully/

ACK with nits fixed.

I've gone ahead and fixed the nits and pushed this. It seems worth having in 0.9.5.

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

--
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]