* [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar... @ 2006-12-23 14:18 Thiemo Seufer 2006-12-23 14:57 ` Fabrice Bellard 0 siblings, 1 reply; 3+ messages in thread From: Thiemo Seufer @ 2006-12-23 14:18 UTC (permalink / raw) To: qemu-devel CVSROOT: /sources/qemu Module name: qemu Changes by: Thiemo Seufer <ths> 06/12/23 14:18:40 Modified files: . : elf_ops.h loader.c hw : mips_r4k.c target-arm : cpu.h target-i386 : cpu.h target-m68k : cpu.h target-mips : cpu.h target-ppc : cpu.h target-sh4 : cpu.h target-sparc : cpu.h Log message: Check ELF binaries for machine type and endianness. CVSWeb URLs: http://cvs.savannah.gnu.org/viewcvs/qemu/elf_ops.h?cvsroot=qemu&r1=1.3&r2=1.4 http://cvs.savannah.gnu.org/viewcvs/qemu/loader.c?cvsroot=qemu&r1=1.3&r2=1.4 http://cvs.savannah.gnu.org/viewcvs/qemu/hw/mips_r4k.c?cvsroot=qemu&r1=1.27&r2=1.28 http://cvs.savannah.gnu.org/viewcvs/qemu/target-arm/cpu.h?cvsroot=qemu&r1=1.17&r2=1.18 http://cvs.savannah.gnu.org/viewcvs/qemu/target-i386/cpu.h?cvsroot=qemu&r1=1.38&r2=1.39 http://cvs.savannah.gnu.org/viewcvs/qemu/target-m68k/cpu.h?cvsroot=qemu&r1=1.1&r2=1.2 http://cvs.savannah.gnu.org/viewcvs/qemu/target-mips/cpu.h?cvsroot=qemu&r1=1.15&r2=1.16 http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/cpu.h?cvsroot=qemu&r1=1.23&r2=1.24 http://cvs.savannah.gnu.org/viewcvs/qemu/target-sh4/cpu.h?cvsroot=qemu&r1=1.3&r2=1.4 http://cvs.savannah.gnu.org/viewcvs/qemu/target-sparc/cpu.h?cvsroot=qemu&r1=1.25&r2=1.26 ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar... 2006-12-23 14:18 [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar Thiemo Seufer @ 2006-12-23 14:57 ` Fabrice Bellard 2006-12-23 15:22 ` Thiemo Seufer 0 siblings, 1 reply; 3+ messages in thread From: Fabrice Bellard @ 2006-12-23 14:57 UTC (permalink / raw) To: qemu-devel Your change in elf_ops.h regarding the labels fail1 and fail2 is not necessary: qemu_free(NULL) is perfectly valid and there are many pieces of code relying on it. Please revert. Regards, Fabrice. Thiemo Seufer wrote: > CVSROOT: /sources/qemu > Module name: qemu > Changes by: Thiemo Seufer <ths> 06/12/23 14:18:40 > > Modified files: > . : elf_ops.h loader.c > hw : mips_r4k.c > target-arm : cpu.h > target-i386 : cpu.h > target-m68k : cpu.h > target-mips : cpu.h > target-ppc : cpu.h > target-sh4 : cpu.h > target-sparc : cpu.h > > Log message: > Check ELF binaries for machine type and endianness. > > CVSWeb URLs: > http://cvs.savannah.gnu.org/viewcvs/qemu/elf_ops.h?cvsroot=qemu&r1=1.3&r2=1.4 > http://cvs.savannah.gnu.org/viewcvs/qemu/loader.c?cvsroot=qemu&r1=1.3&r2=1.4 > http://cvs.savannah.gnu.org/viewcvs/qemu/hw/mips_r4k.c?cvsroot=qemu&r1=1.27&r2=1.28 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-arm/cpu.h?cvsroot=qemu&r1=1.17&r2=1.18 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-i386/cpu.h?cvsroot=qemu&r1=1.38&r2=1.39 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-m68k/cpu.h?cvsroot=qemu&r1=1.1&r2=1.2 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-mips/cpu.h?cvsroot=qemu&r1=1.15&r2=1.16 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-ppc/cpu.h?cvsroot=qemu&r1=1.23&r2=1.24 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-sh4/cpu.h?cvsroot=qemu&r1=1.3&r2=1.4 > http://cvs.savannah.gnu.org/viewcvs/qemu/target-sparc/cpu.h?cvsroot=qemu&r1=1.25&r2=1.26 > > > _______________________________________________ > Qemu-devel mailing list > Qemu-devel@nongnu.org > http://lists.nongnu.org/mailman/listinfo/qemu-devel > > ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar... 2006-12-23 14:57 ` Fabrice Bellard @ 2006-12-23 15:22 ` Thiemo Seufer 0 siblings, 0 replies; 3+ messages in thread From: Thiemo Seufer @ 2006-12-23 15:22 UTC (permalink / raw) To: Fabrice Bellard; +Cc: qemu-devel Fabrice Bellard wrote: > Your change in elf_ops.h regarding the labels fail1 and fail2 is not > necessary: qemu_free(NULL) is perfectly valid and there are many pieces > of code relying on it. Please revert. Changed accordingly. Thiemo ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2006-12-23 15:23 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2006-12-23 14:18 [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar Thiemo Seufer 2006-12-23 14:57 ` Fabrice Bellard 2006-12-23 15:22 ` Thiemo Seufer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).