* [UPATCH] force_sig_info()
@ 2002-09-29 12:52 Oleg Nesterov
2002-09-29 13:07 ` Oleg Nesterov
0 siblings, 1 reply; 2+ messages in thread
From: Oleg Nesterov @ 2002-09-29 12:52 UTC (permalink / raw)
To: linux-kernel; +Cc: Ingo Molnar
Hello.
This is my third attempt:
On Mon, 16 Sep 2002, Oleg Nesterov wrote:
> 2.5.34 introduced this change in force_sig_info()
>
> - return send_sig_info(sig, info, t);
> + return send_sig_info(sig, (void *)1, t);
>
> I beleive, it is wrong, info can carry useful information
> from do_page_fault, traps. And this (info *)1 does not
> prevent send_signal() from allocation of siginfo struct.
> Ingo, could you please clarify?
Call me stupid, but i still believe it is bug.
--- linux-2.5.39/kernel/signal.c~ Sun Sep 29 16:37:08 2002
+++ linux-2.5.39/kernel/signal.c Sun Sep 29 16:37:35 2002
@@ -781,7 +781,7 @@
recalc_sigpending_tsk(t);
spin_unlock_irqrestore(&t->sigmask_lock, flags);
- return send_sig_info(sig, (void *)1, t);
+ return send_sig_info(sig, info, t);
}
static int
Oleg.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [UPATCH] force_sig_info()
2002-09-29 12:52 [UPATCH] force_sig_info() Oleg Nesterov
@ 2002-09-29 13:07 ` Oleg Nesterov
0 siblings, 0 replies; 2+ messages in thread
From: Oleg Nesterov @ 2002-09-29 13:07 UTC (permalink / raw)
To: linux-kernel; +Cc: Ingo Molnar
Hello.
Just browsed marc.theaimsgroup.com, it is fixed already.
Sorry for noise.
Oleg.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2002-09-29 12:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-09-29 12:52 [UPATCH] force_sig_info() Oleg Nesterov
2002-09-29 13:07 ` Oleg Nesterov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®