These two patches fix some clang warnings about use of uninitialized data in linux-user's signal related code for PPC and ARM. The issue in both cases is the same: a code path taken in case of failure was doing 'unlock_user_struct()' with parameters which hadn't yet been set up.
I've marked this as for-1.6 because the patches are simple and I think it's nice to get rid of warnings. However, they're not critical for 1.6: * the existing code will be OK because unlock_user_struct() is a no-op unless DEBUG_REMAP is defined * clang isn't our primary compiler on Linux * configure (kind of inadvertently) disables -Werror for clang * there are other warnings not yet fixed anyhow (most notably all the places which use 'dprintf' as a debug macro despite that being the name of a POSIX specified function) Peter Maydell (2): linux-user/signal.c: PPC: Silence clang uninitialized-use warning linux-user/signal.c: Avoid using uninitialized data in ARM sigreturn linux-user/signal.c | 38 +++++++++++++++++++++----------------- 1 file changed, 21 insertions(+), 17 deletions(-) -- 1.7.9.5