]> granicus.if.org Git - strace/commitdiff
Do not return RVAL_DECODED on exiting path of ioctl decoders
authorEugene Syromyatnikov <evgsyr@gmail.com>
Sat, 26 Aug 2017 22:39:24 +0000 (00:39 +0200)
committerEugene Syromyatnikov <evgsyr@gmail.com>
Mon, 28 Aug 2017 00:33:20 +0000 (02:33 +0200)
* print_sg_req_info.c (decode_sg_req_info): Return 1 instead of
RVAL_DECODED | 1 on exiting path.
* scsi.c (decode_sg_scsi_id): Likewise.
* sg_io_v3.c (decode_response): Likewise.
* sg_io_v4.c (decode_response): Likewise.

print_sg_req_info.c
scsi.c
sg_io_v3.c
sg_io_v4.c

index 9f0249ec50219be717437fb3ab464d21d61cf959..cb34d4fdd1cbd5d5fd9f938d762a189235e1ccfa 100644 (file)
@@ -68,7 +68,7 @@ MPERS_PRINTER_DECL(int, decode_sg_req_info,
                tprintf(", duration=%u}", info.duration);
        }
 
-       return RVAL_DECODED | 1;
+       return 1;
 }
 
 #endif /* HAVE_SCSI_SG_H */
diff --git a/scsi.c b/scsi.c
index 35796201106ea13ef39ccc7fe473303c42faaea7..43b829a325184019cd71320ec051b5296787631f 100644 (file)
--- a/scsi.c
+++ b/scsi.c
@@ -78,7 +78,7 @@ decode_sg_scsi_id(struct tcb *const tcp, const kernel_ulong_t arg)
                        id.h_cmd_per_lun,
                        id.d_queue_depth);
        }
-       return RVAL_DECODED | 1;
+       return 1;
 }
 
 #endif /* HAVE_SCSI_SG_H */
index ebeb66c4979a8db66c99087c5da81e8aec98a698..1be078fb8e2d26ba9efef1514e74bab60baf42b0 100644 (file)
@@ -117,12 +117,12 @@ decode_response(struct tcb *const tcp, const kernel_ulong_t arg)
                if (entering_sg_io->dxfer_direction == SG_DXFER_FROM_DEV)
                        PRINT_FIELD_PTR(", ", *entering_sg_io, dxferp);
                PRINT_FIELD_PTR(", ", *entering_sg_io, sbp);
-               return RVAL_DECODED | 1;
+               return 1;
        }
 
        if (sg_io.interface_id != entering_sg_io->interface_id) {
                PRINT_FIELD_U(" => ", sg_io, interface_id);
-               return RVAL_DECODED | 1;
+               return 1;
        }
 
        if (sg_io.dxfer_direction == SG_DXFER_FROM_DEV ||
@@ -155,7 +155,7 @@ decode_response(struct tcb *const tcp, const kernel_ulong_t arg)
        PRINT_FIELD_U(", ", sg_io, duration);
        PRINT_FIELD_FLAGS(", ", sg_io, info, sg_io_info, "SG_INFO_???");
 
-       return RVAL_DECODED | 1;
+       return 1;
 }
 
 #else /* !HAVE_SCSI_SG_H */
index 68a5cbc897b75101680c64ec862377008bb977dc..48442f2a61b8d3c5b6e36e23cc21643e835668e1 100644 (file)
@@ -111,12 +111,12 @@ decode_response(struct tcb *const tcp, const kernel_ulong_t arg)
                /* print i/o fields fetched on entering syscall */
                PRINT_FIELD_X(", ", *entering_sg_io, response);
                PRINT_FIELD_X(", ", *entering_sg_io, din_xferp);
-               return RVAL_DECODED | 1;
+               return 1;
        }
 
        if (sg_io.guard != entering_sg_io->guard) {
                PRINT_FIELD_U(" => ", sg_io, guard);
-               return RVAL_DECODED | 1;
+               return 1;
        }
 
        PRINT_FIELD_U(", ", sg_io, response_len);
@@ -138,7 +138,7 @@ decode_response(struct tcb *const tcp, const kernel_ulong_t arg)
        PRINT_FIELD_D(", ", sg_io, dout_resid);
        PRINT_FIELD_X(", ", sg_io, generated_tag);
 
-       return RVAL_DECODED | 1;
+       return 1;
 }
 
 #else /* !HAVE_LINUX_BSG_H */