]> granicus.if.org Git - strace/commitdiff
Implement -e trace=/regex option
authorJingPiao Chen <chenjingpiao@gmail.com>
Fri, 14 Apr 2017 04:27:08 +0000 (12:27 +0800)
committerDmitry V. Levin <ldv@altlinux.org>
Fri, 14 Apr 2017 12:18:29 +0000 (12:18 +0000)
* qualify.c: Include <regex.h>.
(qualify_syscall_regex): New function.
(qualify_syscall): Use it.
* strace.1: Document -e trace=/regex option.
* NEWS: Mention -e trace=/regex option.
* tests/regex.test: New test.
* tests/Makefile.am (DECODER_TESTS): Add it.
* tests/options-syntax.test: Add checks for invaild regexp
and for regexp that doesn't match a syscall.

NEWS
qualify.c
strace.1
tests/Makefile.am
tests/options-syntax.test
tests/regex.test [new file with mode: 0755]

diff --git a/NEWS b/NEWS
index f7f64f328f5ddb902cc3a656aca19659a3f8d7ee..06c83594051234133b19e95769a9c0f3b29e76c8 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -13,6 +13,8 @@ Noteworthy changes in release ?.?? (????-??-??)
   * Added -e trace=%clock option for tracing clock_* syscalls.
   * Added -e trace=%statfs option for tracing statfs, statfs64 and statvfs
     syscalls.
+  * Added -e trace=/regex option for filtering syscalls using regular
+    expressions.
   * Implemented decoding of signal mask in rt_sigreturn syscall on alpha, arc,
     arm, avr32, bfin, cris, hppa, m68k, metag, microblaze, mips, nios2, or1k,
     powerpc, powerpc64, riscv, sh, sh64, sparc, sparc64, tile, x86, and xtensa
index 157d3138158b1910628c4f8a19b303ec3ace635c..2362e83f2007c48123afb17f4fdaedfdda3788e8 100644 (file)
--- a/qualify.c
+++ b/qualify.c
@@ -27,6 +27,7 @@
 
 #include "defs.h"
 #include "nsig.h"
+#include <regex.h>
 
 typedef unsigned int number_slot_t;
 #define BITS_PER_SLOT (sizeof(number_slot_t) * 8)
@@ -198,6 +199,48 @@ qualify_syscall_number(const char *s, struct number_set *set)
        return done;
 }
 
+static void
+regerror_msg_and_die(int errcode, const regex_t *preg,
+                    const char *str, const char *pattern)
+{
+       char buf[512];
+
+       regerror(errcode, preg, buf, sizeof(buf));
+       error_msg_and_die("%s: %s: %s", str, pattern, buf);
+}
+
+static bool
+qualify_syscall_regex(const char *s, struct number_set *set)
+{
+       regex_t preg;
+       int rc;
+
+       if ((rc = regcomp(&preg, s, REG_EXTENDED | REG_NOSUB)) != 0)
+               regerror_msg_and_die(rc, &preg, "regcomp", s);
+
+       unsigned int p;
+       bool found = false;
+       for (p = 0; p < SUPPORTED_PERSONALITIES; ++p) {
+               unsigned int i;
+
+               for (i = 0; i < nsyscall_vec[p]; ++i) {
+                       if (!sysent_vec[p][i].sys_name)
+                               continue;
+                       rc = regexec(&preg, sysent_vec[p][i].sys_name,
+                                    0, NULL, 0);
+                       if (rc == REG_NOMATCH)
+                               continue;
+                       else if (rc)
+                               regerror_msg_and_die(rc, &preg, "regexec", s);
+                       add_number_to_set(i, &set[p]);
+                       found = true;
+               }
+       }
+
+       regfree(&preg);
+       return found;
+}
+
 static unsigned int
 lookup_class(const char *s)
 {
@@ -284,6 +327,8 @@ qualify_syscall(const char *token, struct number_set *set)
 {
        if (*token >= '0' && *token <= '9')
                return qualify_syscall_number(token, set);
+       if (*token == '/')
+               return qualify_syscall_regex(token + 1, set);
        return qualify_syscall_class(token, set)
               || qualify_syscall_name(token, set);
 }
index 84ff4ee8667c30dcc7e5412b5656766bdc4d55ee..fc5195c77f8d73215e71857bb685d8dc653e20cc 100644 (file)
--- a/strace.1
+++ b/strace.1
@@ -382,6 +382,14 @@ trace those four system calls.  Be careful when making inferences
 about the user/kernel boundary if only a subset of system calls
 are being monitored.  The default is
 .BR trace = all .
+.TP
+\fB\-e\ trace\fR=/\,\fIregex\fR
+Trace only those system calls that match the
+.IR regex .
+You can use
+.B POSIX
+Extended Regular Expression syntax (see
+.BR regex (7)).
 .PP
 .BR "\-e\ trace" = %file
 .TP
index 3533b595802278a4f1d870de8ac5453fed770e3b..cc71998e9f256ecd2e64b94cacdfd55e64b1a7bf 100644 (file)
@@ -539,6 +539,7 @@ DECODER_TESTS = \
        qual_fault-exit_group.test \
        read-write.test \
        readv.test \
+       regex.test \
        sched.test \
        scm_rights-fd.test \
        seccomp-strict.test \
index 8aa7cf0cd477b3880060d2571cb620839a86ffa3..a9fe710fe54959d57b1935cdad52201e820b679d 100755 (executable)
@@ -39,6 +39,16 @@ check_exit_status_and_stderr()
                        "strace $* failed to print expected diagnostics"
 }
 
+check_exit_status_and_stderr_using_grep()
+{
+       $STRACE "$@" 2> "$LOG" &&
+               dump_log_and_fail_with \
+                       "strace $* failed to handle the error properly"
+       match_grep "$LOG" "$EXP" ||
+               dump_log_and_fail_with \
+                       "strace $* failed to print expected diagnostics"
+}
+
 strace_exp="${STRACE##* }"
 
 check_e()
@@ -50,6 +60,15 @@ __EOF__
        check_exit_status_and_stderr "$@"
 }
 
+check_e_using_grep()
+{
+       local pattern="$1"; shift
+       cat > "$EXP" << __EOF__
+$strace_exp: $pattern
+__EOF__
+       check_exit_status_and_stderr_using_grep "$@"
+}
+
 check_h()
 {
        local pattern="$1"; shift
@@ -72,6 +91,7 @@ check_e "invalid system call '-2'" -e -2
 check_e "invalid system call '-3'" -etrace=-3
 check_e "invalid system call '-4'" -e trace=-4
 check_e "invalid system call '-5'" -e trace=1,-5
+check_e "invalid system call '/non_syscall'" -e trace=/non_syscall
 check_e "invalid system call '2147483647'" -e 2147483647
 check_e "invalid system call '2147483648'" -e 2147483648
 check_e "invalid system call '4294967295'" -e 4294967295
@@ -88,6 +108,12 @@ check_e "invalid descriptor '!'" -ewrite='!'
 check_e "invalid descriptor '!'" -eread='0,!'
 check_e "invalid descriptor '!,'" -ewrite='!,'
 
+check_e_using_grep 'regcomp: \+id: [[:alpha:]].+' -e trace='/+id'
+check_e_using_grep 'regcomp: \*id: [[:alpha:]].+' -e trace='/*id'
+check_e_using_grep 'regcomp: \{id: [[:alpha:]].+' -e trace='/{id'
+check_e_using_grep 'regcomp: \(id: [[:alpha:]].+' -e trace='/(id'
+check_e_using_grep 'regcomp: \[id: [[:alpha:]].+' -e trace='/[id'
+
 check_h 'must have PROG [ARGS] or -p PID'
 check_h 'PROG [ARGS] must be specified with -D' -D -p $$
 check_h '-c and -C are mutually exclusive' -c -C true
diff --git a/tests/regex.test b/tests/regex.test
new file mode 100755 (executable)
index 0000000..0f1c8a2
--- /dev/null
@@ -0,0 +1,74 @@
+#!/bin/sh
+#
+# Check -e trace=/regex option.
+#
+# Copyright (c) 2017 The strace developers.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions
+# are met:
+# 1. Redistributions of source code must retain the above copyright
+#    notice, this list of conditions and the following disclaimer.
+# 2. Redistributions in binary form must reproduce the above copyright
+#    notice, this list of conditions and the following disclaimer in the
+#    documentation and/or other materials provided with the distribution.
+# 3. The name of the author may not be used to endorse or promote products
+#    derived from this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
+# IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
+# OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+# IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
+# INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
+# NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
+# THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+. "${srcdir=.}/init.sh"
+
+while read w s; do {
+       try_run_prog "../$s" || continue
+       run_strace -a$w -e/^clock ../$s > "$EXP"
+       match_diff "$LOG" "$EXP"
+} < /dev/null; done << EOF
+37 clock_adjtime
+40 clock_nanosleep
+36 clock_xettime
+EOF
+
+# Surprised that fcntl*, futimesat, utime, utimensat, and utimes tests
+# linked with musl use clock_gettime?  Me too!
+grep -E -v '^(#|clock_|times$|fcntl|futimesat$|utime)' \
+       < "$srcdir/pure_executables.list" > negative.list
+
+while read s; do {
+       try_run_prog "../$s" || continue
+       run_strace -qq -esignal=none -e/^clock ../$s > /dev/null
+       match_diff "$LOG" /dev/null
+} < /dev/null; done < negative.list
+
+while read w s; do {
+       try_run_prog "../$s" || continue
+       run_strace -a$w -e/^sched_ ../$s > "$EXP"
+       match_diff "$LOG" "$EXP"
+} < /dev/null; done << EOF
+28 sched_xetaffinity
+23 sched_xetparam
+31 sched_rr_get_interval
+33 sched_get_priority_mxx
+29 sched_xetattr
+22 sched_xetscheduler
+14 sched_yield
+EOF
+
+grep -E -v '^(#|sched_|times$)' \
+       < "$srcdir/pure_executables.list" > negative.list
+
+while read s; do {
+       try_run_prog "../$s" || continue
+       run_strace -qq -esignal=none -e/^sched_ ../$s > /dev/null
+       match_diff "$LOG" /dev/null
+} < /dev/null; done < negative.list