[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/tests/syscall
On Sun, May 29, 2011 at 02:57:14PM +0100, Julio M. Merino Vidal wrote:
> On 5/29/11 1:57 PM, Matthias Scheler wrote:
> >Modified Files:
> > src/tests/syscall: t_pollts.c
> >Log Message:
> >Also use ATF_REQUIRE_EQ_MSG() instead of ATF_REQUIRE_MSG() to avoid
> >crashes if one of the required conditions isn't met.
> It could be a bug in the macro. I tried changing the last
> ATF_REQUIRE_EQ_MSG call in the program to ATF_REQUIRE_MSG but it did
> not fail. And I tried adding an explicitly-failing ATF_REQUIRE_MSG
> and did not see a crash either...
I don't think it's a bug in the macro. The problem was that it use the
0 value as the format string which caused a crash. I'm not sure
why I haven't seen this before.
Matthias Scheler http://zhadum.org.uk/
Main Index |
Thread Index |