2005-11-08 Frank Ch. Eigler <fche@redhat.com>

	* tapsets.cxx (blacklisted_p): Tolerate NULL filename parameter.
	* src/testsuite/semok/twenty.stp: New test to enumerate everything
	dwarfly probeable.

diff --git a/ChangeLog b/ChangeLog
index ef0fca5b5933f7521c1f196fc55e5935d92ed70c..8dea6bdb7adec763dc75ccb2ec95f8019063759a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2005-11-08  Frank Ch. Eigler  <fche@redhat.com>
+
+       * tapsets.cxx (blacklisted_p): Tolerate NULL filename parameter.
+       * src/testsuite/semok/twenty.stp: New test to enumerate everything
+       dwarfly probeable.
+
 2005-11-08  Frank Ch. Eigler  <fche@redhat.com>
 
        * translate.cxx (c_unparser::visit_foreach_loop): With PR 1275

diff --git a/tapsets.cxx b/tapsets.cxx
index 7b0d2b78797ed8a5285c36370f4324f6b1cd335d..d9a75340b27be140762f8a8279b5701e1615b09c 100644 (file)
--- a/tapsets.cxx
+++ b/tapsets.cxx
@@ -1710,7 +1710,8 @@ dwarf_query::blacklisted_p(string const & funcname,
   // properly generalized, perhaps via a table populated from script
   // files.  A "noprobe kernel.function("...")"  construct might do
   // the trick.
-  string filename_s = filename ? filename : ""; // is passed as const char*
+  if (filename == 0) filename = ""; // possibly 0
+  string filename_s = filename; // is passed as const char*
   if (funcname == "do_IRQ" ||
       filename_s == "kernel/kprobes.c" ||
       0 == fnmatch ("arch/*/kernel/kprobes.c", filename, 0) ||

diff --git a/testsuite/semok/twenty.stp b/testsuite/semok/twenty.stp
new file mode 100755 (executable)
index 0000000..6d7f518
--- /dev/null
+++ b/testsuite/semok/twenty.stp
@@ -0,0 +1,8 @@
+#! stap -p2
+
+probe kernel.function("*") {}
+probe module("*").function("*") {}
+probe kernel.function("*").return {}
+probe module("*").function("*").return {}
+probe kernel.inline("*") {}
+probe module("*").inline("*") {}