On Fri, Jul 23, 2010 at 10:56:58AM -0700, Garrett Cooper wrote:
> On Fri, Jul 23, 2010 at 10:17 AM, Nicolas Joly <[email protected]> wrote:
> >
> > Hi,
> >
> > The attached patch do some cleanup in the abort01 testcase ...
> >
> > 1) The attempt to remove the generated core is wrong. The hard-coded
> > `core' name is bad and the test temporary directory removal already
> > take care of this.
> >
> > 2) Make the test fail gracefully if the running environment does not
> > allow generating core files.
> >
> > nj...@lanfeust [syscalls/abort]> ./abort01
> > abort01 1 TPASS : Test passed
> > nj...@lanfeust [syscalls/abort]> (ulimit -c 0 && ./abort01)
> > abort01 1 TCONF : core file size limit must be greater than 0.
> >
> > Signed-off-by: Nicolas Joly <[email protected]>
>
> Looks ok, but is there any particular reason why you removed the unlink(2)
> call?
The `unlink("core")' one ?
Well, at least RHEL do use `proc.<pid>' when generating core file
names ... Not speaking about admins that might have configured it to
use another scheme.
IMO, relying on a specific name, for a configurable system, is a bad
idea; and can only lead to problems.
By example, this test from mkdir09.c cannot succeed on most RHEL
systems:
/* Check for core file in test directory. */
if (access("core", 0) == 0) {
tst_resm(TWARN, "\tCore file found in test directory.");
tst_exit();
}
--
Nicolas Joly
Biological Software and Databanks.
Institut Pasteur, Paris.
------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
