Re: [Qemu-devel] [PULL v2 0/8] target-mips queue

[Peter Maydell]

On 22 February 2017 at 00:21, Yongbok Kim <address@hidden> wrote:
> Hi,
>
> This is the v2 of the pull-req for target-mips.
> The warning treated as error has been fixed and the file is now isolated for 
> mips64el only.
> Boston has further update to have default -drive to if=ide.
>
> Regards,
> Yongbok
>
> The following changes since commit 796b288f7be875045670f963ce99991b3c8e96ac:
>
>   Merge remote-tracking branch 'remotes/cody/tags/block-pull-request' into 
> staging (2017-02-21 15:48:22 +0000)
>
> are available in the git repository at:
>
>   git://github.com/yongbok/upstream-qemu.git tags/mips-20170222
>
> for you to fetch changes up to d3473e147a754e999718bf6fcb015d9978c6a1ee:
>
>   hw/mips: MIPS Boston board support (2017-02-21 23:49:30 +0000)
>
> ----------------------------------------------------------------
> MIPS patches 2017-02-22
>
> Changes:
> * Add MIPS Boston board support

This seems to cause a lot of new clang sanitizer warnings:
/home/petmay01/linaro/qemu-for-merges/dtc/libfdt/fdt_rw.c:288:21:
runtime error: null pointer passed as argument 2, which is declared to
never be null

which are all for the memcpy() in fdt_setprop() when fdt_setprop()
is passed a NULL pointer.

On the other hand the implementation of fdt_setprop() hasn't changed
in years so I'm not quite sure why this warning is only showing up
now. It may be that the clang build used to pull in system libfdt
but now has built its own.

On that basis I've pushed this to master and we'll figure out
the sanitizer issues separately.

Please make sure you update the 'Build dependencies' section of
http://wiki.qemu-project.org/ChangeLog/2.9
to flag up the new libfdt version dependency to users.

thanks
-- PMM