|
 |
132810 |
From 99ee8b19901f4908e2a2942731c34e03aadd9549 Mon Sep 17 00:00:00 2001
|
|
|
132810 |
From: Serhei Makarov <smakarov@redhat.com>
|
|
|
132810 |
Date: Tue, 30 Oct 2018 17:10:53 -0400
|
|
|
132810 |
Subject: [PATCH 13/32] bpf-translate.cxx :: fix segfault with malformed
|
|
|
132810 |
register
|
|
|
132810 |
|
|
|
132810 |
---
|
|
|
132810 |
bpf-translate.cxx | 9 +++++++--
|
|
|
132810 |
testsuite/systemtap.bpf/asm_tests/err-regparse.stp | 9 +++++++++
|
|
|
132810 |
2 files changed, 16 insertions(+), 2 deletions(-)
|
|
|
132810 |
create mode 100644 testsuite/systemtap.bpf/asm_tests/err-regparse.stp
|
|
|
132810 |
|
|
|
132810 |
diff --git a/bpf-translate.cxx b/bpf-translate.cxx
|
|
|
132810 |
index bb133bae5..d46dae44a 100644
|
|
|
132810 |
--- a/bpf-translate.cxx
|
|
|
132810 |
+++ b/bpf-translate.cxx
|
|
|
132810 |
@@ -952,8 +952,13 @@ bpf_unparser::emit_asm_arg (const asm_stmt &stmt, const std::string &arg,
|
|
|
132810 |
{
|
|
|
132810 |
/* arg is a register number */
|
|
|
132810 |
std::string reg = arg[0] == 'r' ? arg.substr(1) : arg;
|
|
|
132810 |
- unsigned long num = stoul(reg, 0, 0);
|
|
|
132810 |
- if (num > 10)
|
|
|
132810 |
+ unsigned long num;
|
|
|
132810 |
+ bool parsed = false;
|
|
|
132810 |
+ try {
|
|
|
132810 |
+ num = stoul(reg, 0, 0);
|
|
|
132810 |
+ parsed = true;
|
|
|
132810 |
+ } catch (std::exception &e) {} // XXX: invalid_argument, out_of_range
|
|
|
132810 |
+ if (!parsed || num > 10)
|
|
|
132810 |
throw SEMANTIC_ERROR (_F("invalid bpf register '%s'",
|
|
|
132810 |
arg.c_str()), stmt.tok);
|
|
|
132810 |
return this_prog.lookup_reg(num);
|
|
|
132810 |
diff --git a/testsuite/systemtap.bpf/asm_tests/err-regparse.stp b/testsuite/systemtap.bpf/asm_tests/err-regparse.stp
|
|
|
132810 |
new file mode 100644
|
|
|
132810 |
index 000000000..ba66800e6
|
|
|
132810 |
--- /dev/null
|
|
|
132810 |
+++ b/testsuite/systemtap.bpf/asm_tests/err-regparse.stp
|
|
|
132810 |
@@ -0,0 +1,9 @@
|
|
|
132810 |
+function foo:long () %{ /* bpf */ /* pure */
|
|
|
132810 |
+ 0xb7, $rc, -, -, 50; /* mov $rc, 50 */
|
|
|
132810 |
+ 0xbf, $$, rc, -, -; /* mov $$, $rc -- TYPO */
|
|
|
132810 |
+%}
|
|
|
132810 |
+
|
|
|
132810 |
+probe begin {
|
|
|
132810 |
+ printf("foo()=%d should be fifty\n", foo())
|
|
|
132810 |
+ exit()
|
|
|
132810 |
+}
|
|
|
132810 |
--
|
|
|
132810 |
2.14.5
|
|
|
132810 |
|