diff options
| author | Peter Maydell <peter.maydell@linaro.org> | 2018-12-13 12:02:52 +0000 |
|---|---|---|
| committer | Cornelia Huck <cohuck@redhat.com> | 2018-12-20 17:07:24 +0100 |
| commit | 55281a2c53b884d0c2bce31aee592faf20a28eed (patch) | |
| tree | 4b7886dd04456b95c83ef1ebeefd501aff6ba094 /include/hw/misc/imx7_gpr.h | |
| parent | b72566a4ffaddbc0c0c1f6f5ee91b42ab13ff429 (diff) | |
| download | qemu-55281a2c53b884d0c2bce31aee592faf20a28eed.zip qemu-55281a2c53b884d0c2bce31aee592faf20a28eed.tar.gz qemu-55281a2c53b884d0c2bce31aee592faf20a28eed.tar.bz2 | |
hw/s390/ccw.c: Don't take address of packed members
Taking the address of a field in a packed struct is a bad idea, because
it might not be actually aligned enough for that pointer type (and
thus cause a crash on dereference on some host architectures). Newer
versions of clang warn about this.
Avoid the problem by using local copies of the PMCW and SCSW
struct fields in copy_schib_from_guest() and copy_schib_to_guest().
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20181213120252.21697-1-peter.maydell@linaro.org>
Reviewed-by: Farhan Ali <alifm@linux.ibm.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
Diffstat (limited to 'include/hw/misc/imx7_gpr.h')
0 files changed, 0 insertions, 0 deletions