diff options
author | avg <avg@FreeBSD.org> | 2013-03-23 08:52:50 +0000 |
---|---|---|
committer | avg <avg@FreeBSD.org> | 2013-03-23 08:52:50 +0000 |
commit | 036eb5da00d711b07697be4c49fbe8dec348d185 (patch) | |
tree | cf75f7ede352b4415fb109dc24473e6c0ed82f77 /sys/cddl/dev/fbt | |
parent | 2d50e830b3e79fa42f303add8a99872897abdc2a (diff) | |
download | FreeBSD-src-036eb5da00d711b07697be4c49fbe8dec348d185.zip FreeBSD-src-036eb5da00d711b07697be4c49fbe8dec348d185.tar.gz |
fbt_getargdesc: correctly handle types for return probes
MFC after: 6 days
Diffstat (limited to 'sys/cddl/dev/fbt')
-rw-r--r-- | sys/cddl/dev/fbt/fbt.c | 22 |
1 files changed, 17 insertions, 5 deletions
diff --git a/sys/cddl/dev/fbt/fbt.c b/sys/cddl/dev/fbt/fbt.c index 93bd118..b828163 100644 --- a/sys/cddl/dev/fbt/fbt.c +++ b/sys/cddl/dev/fbt/fbt.c @@ -1260,6 +1260,11 @@ fbt_getargdesc(void *arg __unused, dtrace_id_t id __unused, void *parg, dtrace_a uint32_t offset; ushort_t info, kind, n; + if (fbt->fbtp_roffset != 0 && desc->dtargd_ndx == 0) { + (void) strcpy(desc->dtargd_native, "int"); + return; + } + desc->dtargd_ndx = DTRACE_ARGNONE; /* Get a pointer to the CTF data and it's length. */ @@ -1310,12 +1315,19 @@ fbt_getargdesc(void *arg __unused, dtrace_id_t id __unused, void *parg, dtrace_a return; } - /* Check if the requested argument doesn't exist. */ - if (ndx >= n) - return; + if (fbt->fbtp_roffset != 0) { + /* Only return type is available for args[1] in return probe. */ + if (ndx > 1) + return; + ASSERT(ndx == 1); + } else { + /* Check if the requested argument doesn't exist. */ + if (ndx >= n) + return; - /* Skip the return type and arguments up to the one requested. */ - dp += ndx + 1; + /* Skip the return type and arguments up to the one requested. */ + dp += ndx + 1; + } if (fbt_type_name(&lc, *dp, desc->dtargd_native, sizeof(desc->dtargd_native)) > 0) desc->dtargd_ndx = ndx; |