]> git.ozlabs.org Git - ccan/blobdiff - ccan/read_write_all/test/run-read_all.c
Mark unused arguments in many modules.
[ccan] / ccan / read_write_all / test / run-read_all.c
index 5fa0ec392920d119af1b2d963238cf4b188ea109..6e6588d4149a79351f04993bbc37470ab8d36626 100644 (file)
@@ -1,8 +1,8 @@
 /* FIXME: Do something tricky to ensure we really do loop in read_all. */
 
-#include "read_write_all/read_write_all.h"
-#include "read_write_all/read_write_all.c"
-#include "tap/tap.h"
+#include <ccan/read_write_all/read_write_all.h>
+#include <ccan/read_write_all/read_write_all.c>
+#include <ccan/tap/tap.h>
 #include <unistd.h>
 #include <sys/types.h>
 #include <signal.h>
@@ -17,6 +17,7 @@ static int p2c[2], c2p[2];
 static void got_signal(int sig)
 {
        char c = 0;
+       (void)sig;
        if (write(p2c[1], &c, 1) == 1)
                sigcount++;
 }
@@ -24,13 +25,14 @@ static void got_signal(int sig)
 /* < PIPE_BUF *will* be atomic.  But > PIPE_BUF only *might* be non-atomic. */
 #define BUFSZ (1024*1024)
 
-int main(int argc, char *argv[])
+int main(void)
 {
-       char buffer[BUFSZ*2] = { 0 };
+       char *buffer;
        char c = 0;
        int status;
        pid_t child;
 
+       buffer = calloc(BUFSZ, 2);
        plan_tests(6);
 
        /* We fork and torture parent. */
@@ -44,7 +46,7 @@ int main(int argc, char *argv[])
                /* Child.  Make sure parent ready, then write in two parts. */
                if (read(p2c[0], &c, 1) != 1)
                        exit(1);
-               memset(buffer, 0xff, sizeof(buffer));
+               memset(buffer, 0xff, BUFSZ*2);
                if (!write_all(c2p[1], buffer, BUFSZ))
                        exit(2);
                if (kill(getppid(), SIGUSR1) != 0)
@@ -63,12 +65,13 @@ int main(int argc, char *argv[])
        close(c2p[1]);
        signal(SIGUSR1, got_signal);
        ok1(write(p2c[1], &c, 1) == 1);
-       ok1(read_all(c2p[0], buffer, sizeof(buffer)));
-       ok1(memchr(buffer, 0, sizeof(buffer)) == NULL);
+       ok1(read_all(c2p[0], buffer, BUFSZ*2));
+       ok1(memchr(buffer, 0, BUFSZ*2) == NULL);
        ok1(sigcount == 1);
        ok1(wait(&status) == child);
        ok(WIFEXITED(status) && WEXITSTATUS(status) == 0,
           "WIFEXITED(status) = %u, WEXITSTATUS(status) = %u",
           WIFEXITED(status), WEXITSTATUS(status));
+       free(buffer);
        return exit_status();
 }