[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Please see my comments below about the real time signal bugs.
By the way, do you have any plans to commit this part?
I can change the testing program to be a unit test.
2016-08-03 0:10 GMT-07:00 Christos Zoulas <christos%astron.com@localhost>:
> In article <CA+SXE9v9C=SfjQxZRCXLBSpKg=
> Charles Cui <charles.cui1984%gmail.com@localhost> wrote:
> >Hi Christos,
> > I spend time in studying the current code base, and checkout to
> >commit to see where I introduced the problems (just like a binary search).
> >In this way, I hope to narrow down to the real problem.
> >One thing that I found was my sshd is still working even applying all
> >I can login into the vm via ssh, and run tests.
> >By the way, how do you fix the problems which makes your sshd broken?
> 1. In siggetinfo(), if we exit the loop without finding a signal, count ==
> and we return right there without initializing "out". This is what
> things for me.
yeah, this is a bug. I looked at the code several times, but never noticed
this problem :)
> 2. In sigget we only remove from the set if ret == 1 (but this is not a bug
> really); I think it is better to do (more similar to the old code):
> sigdelset(&sp->sp_set, signo);
> count = siggetinfo(sp, out, signo);
> if (count > 1)
> sigaddset(&sp->sp_set, signo);
got it, this looks better.
Main Index |
Thread Index |