summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorhunt <hunt>2007-07-26 17:11:51 +0000
committerhunt <hunt>2007-07-26 17:11:51 +0000
commit39dd928d212737c6b5c7e42ed71a0c699c0faf11 (patch)
treeab76b4df3595152b7248c28c50070fd0c146d93f
parent9ddaa7a839385ea194d958295ac330350147c52c (diff)
downloadsystemtap-steved-39dd928d212737c6b5c7e42ed71a0c699c0faf11.tar.gz
systemtap-steved-39dd928d212737c6b5c7e42ed71a0c699c0faf11.tar.xz
systemtap-steved-39dd928d212737c6b5c7e42ed71a0c699c0faf11.zip
2007-07-26 Martin Hunt <hunt@redhat.com>
Spotted by Ming Chang. * procfs.c (_stp_register_ctl_channel): Fix iterator when removing proc entries when an error occurs.
-rw-r--r--runtime/transport/ChangeLog12
-rw-r--r--runtime/transport/procfs.c2
2 files changed, 13 insertions, 1 deletions
diff --git a/runtime/transport/ChangeLog b/runtime/transport/ChangeLog
index b34dc294..7ce5be44 100644
--- a/runtime/transport/ChangeLog
+++ b/runtime/transport/ChangeLog
@@ -1,3 +1,15 @@
+2007-07-26 Martin Hunt <hunt@redhat.com>
+
+ Spotted by Ming Chang.
+ * procfs.c (_stp_register_ctl_channel): Fix iterator when removing
+ proc entries when an error occurs.
+
+2007-07-10 Michal Schmidt <mschmidt@redhat.com>
+
+ * symbols.c (_stp_del_module): Split freeing of module memory into
+ a separate function _stp_free_module().
+ (_stp_do_module): Free module memory if _stp_ins_module() fails.
+
2007-07-09 Martin Hunt <hunt@redhat.com>
* symbols.c (_stp_ins_module): Check for overflow of
diff --git a/runtime/transport/procfs.c b/runtime/transport/procfs.c
index c6e3f46a..36920707 100644
--- a/runtime/transport/procfs.c
+++ b/runtime/transport/procfs.c
@@ -406,7 +406,7 @@ err1:
for_each_cpu(j) {
if (j == i)
break;
- sprintf(buf, "%d", i);
+ sprintf(buf, "%d", j);
remove_proc_entry (buf, _stp_proc_mod);
}