From: "Jungsoo Son" <jungsoo.son@lge.com>
To: ltp-list@lists.sourceforge.net
Subject: [LTP] [PATCH] syscalls/chown01: Fix test file creation
Date: Sat, 8 Jun 2013 11:54:26 +0900 [thread overview]
Message-ID: <00f801ce63f3$7d4cc8f0$77e65ad0$@lge.com> (raw)
There is a bug in the code to create the file and the code is
very intricate. So replace it with SAFE_FILE_PRINTF().
Signed-off-by: jungsoo.son <jungsoo.son@lge.com>
---
testcases/kernel/syscalls/chown/chown01.c | 24 +++++++++---------------
1 file changed, 9 insertions(+), 15 deletions(-)
diff --git a/testcases/kernel/syscalls/chown/chown01.c
b/testcases/kernel/syscalls/chown/chown01.c
index 50b9ea5..14b699b 100644
--- a/testcases/kernel/syscalls/chown/chown01.c
+++ b/testcases/kernel/syscalls/chown/chown01.c
@@ -116,9 +116,7 @@
#include <signal.h>
#include "test.h"
#include "usctest.h"
-
-void setup();
-void cleanup();
+#include "safe_macros.h"
char *TCID = "chown01";
int TST_TOTAL = 1;
@@ -126,8 +124,10 @@ int TST_TOTAL = 1;
int exp_enos[] = { 0, 0 };
char fname[255];
-int fd, uid, gid;
-char *buf = "davef";
+int uid, gid;
+
+static void setup();
+static void cleanup();
int main(int ac, char **av)
{
@@ -163,7 +163,7 @@ int main(int ac, char **av)
}
-void setup()
+static void setup()
{
tst_sig(NOFORK, DEF_HANDLER, cleanup);
@@ -176,17 +176,11 @@ void setup()
gid = getegid();
sprintf(fname, "t_%d", getpid());
- if ((fd = open(fname, O_RDWR | O_CREAT, 0700)) == -1)
- tst_brkm(TBROK | TERRNO, cleanup,
- "open(%s, O_RDWR|O_CREAT,0700) failed", fname);
- else if (write(fd, &buf, strlen(buf)) == -1)
- tst_brkm(TBROK | TERRNO, cleanup,
- "write(%s, &buf, strlen(buf)) failed", fname);
- else if (close(fd) == -1)
- tst_brkm(TBROK | TERRNO, cleanup, "close(%s) failed",
fname);
+
+ SAFE_FILE_PRINTF(cleanup, fname, "davef");
}
-void cleanup()
+static void cleanup()
{
TEST_CLEANUP;
--
1.7.9.5
------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. A cloud service to automate IT design, transition and operations
2. Dashboards that offer high-level views of enterprise services
3. A single system of record for all IT processes
http://p.sf.net/sfu/servicenow-d2d-j
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
next reply other threads:[~2013-06-08 2:54 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-08 2:54 Jungsoo Son [this message]
2013-06-10 14:07 ` [LTP] [PATCH] syscalls/chown01: Fix test file creation Jan Stancek
2013-06-10 23:36 ` Jungsoo Son
2013-06-11 0:07 ` Wanlong Gao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='00f801ce63f3$7d4cc8f0$77e65ad0$@lge.com' \
--to=jungsoo.son@lge.com \
--cc=ltp-list@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox