Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
um: chan_user: Fix winch_tramp() return value
commit 57ae0b6 upstream. The previous fix here was only partially correct, it did result in returning a proper error value in case of error, but it also clobbered the pid that we need to return from this function (not just zero for success). As a result, it returned 0 here, but later this is treated as a pid and used to kill the process, but since it's now 0 we kill(0, SIGKILL), which makes UML kill itself rather than just the helper thread. Fix that and make it more obvious by using a separate variable for the pid. Fixes: ccf1236 ("um: fix error return code in winch_tramp()") Reported-and-tested-by: Nathan Chancellor <[email protected]> Signed-off-by: Johannes Berg <[email protected]> Cc: [email protected] Signed-off-by: Richard Weinberger <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
- Loading branch information