[Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar...

[Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar...

[Thiemo Seufer]

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

Re: [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar...

[Fabrice Bellard]

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

Re: [Qemu-devel] qemu elf_ops.h loader.c hw/mips_r4k.c target-ar...

[Thiemo Seufer]

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