Implement rt_sigqueueinfo() system call.

[emaste]

Looks fine to me

sys/compat/linux/linux_signal.c

718–719 ↗

(On Diff #2177)

Should we log and/or disallow this case?

[dchagin]

sys/compat/linux/linux_signal.c

718–719 ↗

(On Diff #2177)

No, RT signals. LINUX_SIG_VALID (707-708) validate user supplied sigcode.

[dchagin]

constify l_siginfo_t, merge with D1044 changes.

[trasz]

sys/compat/linux/linux_signal.c

795 ↗

(On Diff #4374)

I think you're missing a lot of access checks here. See sys/kern/kern_sig:sys_kill(); in particular the IN_CAPABILITY_MODE() and p_cansignal().

[trasz]

sys/compat/linux/linux_signal.c

795 ↗

(On Diff #4374)

Erm, ignore the IN_CAPABILITY_MODE(), but p_cansignal() should still be used, no?

[dchagin]

Yes, you are right. fixed, thanks!

[dchagin]

typo fixed.

[dchagin]

sys/compat/linux/linux_signal.c

760 ↗

(On Diff #4407)

Linux allows SI_ here. So, I plan to convert this later.

[trasz]

I'd consider refactoring the code, to break this code into separate function:

linux_copyin_siginfo(...) {

error = copyin(args->info, &linfo, sizeof(linfo));
if (error != 0)
        return (error);

 ksiginfo_init(&ksi);
  lsiginfo_to_ksiginfo(&linfo, &ksi, sig);

}

... and then do something similar in a lot of other cases when you have this "copyin and convert", or "convert and copyout" pattern. Still, probably better leave this for another time.