diff options
| author | Josh Stone <jistone@redhat.com> | 2009-02-09 09:28:36 -0800 |
|---|---|---|
| committer | Josh Stone <jistone@redhat.com> | 2009-02-09 09:44:55 -0800 |
| commit | a903021435a22c55a28e470e342b5fce4de98738 (patch) | |
| tree | 5752e318116bf9cfba4a879619223e10e78b7698 | |
| parent | 22f971e83eddada009053d964a397550807b9b50 (diff) | |
| download | systemtap-steved-a903021435a22c55a28e470e342b5fce4de98738.tar.gz systemtap-steved-a903021435a22c55a28e470e342b5fce4de98738.tar.xz systemtap-steved-a903021435a22c55a28e470e342b5fce4de98738.zip | |
process.create: fix the new_pid read order
* process.stp (process.create): Read the task pid *after*
the task pointer is checked for errors.
This is to fix a problem reported on the mailing list:
http://sources.redhat.com/ml/systemtap/2009-q1/msg00403.html
CC: Luis Fernando Muñoz Mejías <Luis.Fernando.Munoz.Mejias@cern.ch>
| -rw-r--r-- | tapset/ChangeLog | 5 | ||||
| -rw-r--r-- | tapset/process.stp | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/tapset/ChangeLog b/tapset/ChangeLog index 570d3697..1aebe30c 100644 --- a/tapset/ChangeLog +++ b/tapset/ChangeLog @@ -1,3 +1,8 @@ +2009-02-09 Josh Stone <jistone@redhat.com> + + * process.stp (process.create): Read the task pid *after* + the task pointer is checked for errors. + 2009-02-04 K Prasad <prasad@linux.vnet.ibm.com> PR 7030. diff --git a/tapset/process.stp b/tapset/process.stp index b28a72cd..7aab9379 100644 --- a/tapset/process.stp +++ b/tapset/process.stp @@ -25,8 +25,8 @@ function _IS_ERR:long(ptr:long) %{ /* pure */ */ probe process.create = kernel.function("copy_process").return { task = $return - new_pid = task_pid(task) if (_IS_ERR(task)) next + new_pid = task_pid(task) } |