]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
fstests: fix compilation error in splice2pipe on old (<C99) compilers
authorAndrey Albershteyn <aalbersh@redhat.com>
Tue, 16 May 2023 10:23:03 +0000 (12:23 +0200)
committerZorro Lang <zlang@kernel.org>
Sat, 27 May 2023 18:03:49 +0000 (02:03 +0800)
Compilation fails on system with compiler which uses older C dialect
(e.g. RHEL7; gcc with gnu90) with:

splice2pipe.c: In function 'prepare_pipe':
splice2pipe.c:54:2: error: 'for' loop initial declarations are only allowed in C99 mode
  for (unsigned r = pipe_size; r > 0;) {

Fix it by declaring 'r' outside of the loop.

Signed-off-by: Andrey Albershteyn <aalbersh@redhat.com>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Zorro Lang <zlang@kernel.org>
src/splice2pipe.c

index bd33ff6736da3392c53758b3849aba2be286ca61..00b3a23af993ae361983d33be14409243c51b474 100644 (file)
@@ -41,6 +41,7 @@
  */
 static void prepare_pipe(int p[2])
 {
+       unsigned int r = 0;
        if (pipe(p)) {
                perror("pipe failed");
                abort();
@@ -51,7 +52,7 @@ static void prepare_pipe(int p[2])
 
        /* fill the pipe completely; each pipe_buffer will now have
           the PIPE_BUF_FLAG_CAN_MERGE flag */
-       for (unsigned r = pipe_size; r > 0;) {
+       for (r = pipe_size; r > 0;) {
                unsigned n = r > sizeof(buffer) ? sizeof(buffer) : r;
                write(p[1], buffer, n);
                r -= n;
@@ -59,7 +60,7 @@ static void prepare_pipe(int p[2])
 
        /* drain the pipe, freeing all pipe_buffer instances (but
           leaving the flags initialized) */
-       for (unsigned r = pipe_size; r > 0;) {
+       for (r = pipe_size; r > 0;) {
                unsigned n = r > sizeof(buffer) ? sizeof(buffer) : r;
                read(p[0], buffer, n);
                r -= n;