Hi!
> diff --git a/testcases/kernel/fs/fsstress/fsstress.c
> b/testcases/kernel/fs/fsstress/fsstress.c
> index e3b48ea..37956f3 100644
> --- a/testcases/kernel/fs/fsstress/fsstress.c
> +++ b/testcases/kernel/fs/fsstress/fsstress.c
> @@ -1826,7 +1826,13 @@ dread_f(int opno, long r)
> len = align;
> else if (len > diob.d_maxiosz)
> len = diob.d_maxiosz;
> - buf = memalign(diob.d_mem, len);
> + if (posix_memalign((void **)&buf, diob.d_mem, len) != 0) {
> + perror("posix_memalign");
> + exit (1);
> + } else if (buf == NULL) {
> + fprintf(stderr, "buf remains NULL unexpectly\n");
> + exit (1);
> + }
Okay, last comments:
* exit is a function so we should write exit(1);
* there is no need for the second if to be in else branch as the first
one calls exit anyway.
--
Cyril Hrubis
[email protected]
------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
