linux-user: fix pidfd_send_signal() #24

2023-05-15T15:23:41+02:00

andreas-schwab commented

2023-05-15 15:23:41 +02:00

(Migrated from github.com)

According to pidfd_send_signal(2), info argument can be a NULL pointer.
Fix strace to correctly manage ending comma in parameters.

Fixes: cc054c6f13 ("linux-user: Add pidfd_open(), pidfd_send_signal() and pidfd_getfd() syscalls")
cc: Helge Deller deller@gmx.de
Signed-off-by: Laurent Vivier laurent@vivier.eu
Reviewed-by: Helge Deller deller@gmx.de
Message-Id: 20221005163826.1455313-1-laurent@vivier.eu
Signed-off-by: Laurent Vivier laurent@vivier.eu
(cherry picked from commit 46187d707e)

According to pidfd_send_signal(2), info argument can be a NULL pointer. Fix strace to correctly manage ending comma in parameters. Fixes: cc054c6f13 ("linux-user: Add pidfd_open(), pidfd_send_signal() and pidfd_getfd() syscalls") cc: Helge Deller <deller@gmx.de> Signed-off-by: Laurent Vivier <laurent@vivier.eu> Reviewed-by: Helge Deller <deller@gmx.de> Message-Id: <20221005163826.1455313-1-laurent@vivier.eu> Signed-off-by: Laurent Vivier <laurent@vivier.eu> (cherry picked from commit 46187d707e7639b743a3b9f72da03ad4b9abc255)

dfaggioli (Migrated from github.com) approved these changes 2023-05-16 05:31:45 +02:00

dfaggioli (Migrated from github.com) left a comment

Not sure why automatic checks didn't run properly, but I checked manually and this is fine.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dfaggioli/qemu#24