Page MenuHomeFreeBSD
Files F162290587

D57979.id181354.diff
No OneTemporary
Actions

D57979.id181354.diff
View Options

diff --git a/sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c b/sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c
--- a/sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c
+++ b/sys/cddl/contrib/opensolaris/uts/common/dtrace/dtrace.c
@@ -16260,7 +16260,8 @@
}
if (prb_sec->dofs_entsize == 0 ||
- prb_sec->dofs_entsize > prb_sec->dofs_size) {
+ prb_sec->dofs_entsize > prb_sec->dofs_size ||
+ prb_sec->dofs_entsize < sizeof (dof_probe_t)) {
dtrace_dof_error(dof, "invalid entry size");
return (-1);
}
@@ -16285,6 +16286,11 @@
return (-1);
}
+ if (enoff_sec != NULL && enoff_sec->dofs_entsize != sizeof (uint32_t)) {
+ dtrace_dof_error(dof, "invalid entry size");
+ return (-1);
+ }
+
arg = (uint8_t *)(uintptr_t)(daddr + arg_sec->dofs_offset);
nprobes = prb_sec->dofs_size / prb_sec->dofs_entsize;
@@ -16325,7 +16331,7 @@
if (probe->dofpr_offidx + probe->dofpr_noffs <
probe->dofpr_offidx ||
(probe->dofpr_offidx + probe->dofpr_noffs) *
- off_sec->dofs_entsize > off_sec->dofs_size) {
+ (uint64_t) off_sec->dofs_entsize > off_sec->dofs_size) {
dtrace_dof_error(dof, "invalid probe offset");
return (-1);
}
@@ -16347,7 +16353,8 @@
} else if (probe->dofpr_enoffidx +
probe->dofpr_nenoffs < probe->dofpr_enoffidx ||
(probe->dofpr_enoffidx + probe->dofpr_nenoffs) *
- enoff_sec->dofs_entsize > enoff_sec->dofs_size) {
+ (uint64_t) enoff_sec->dofs_entsize >
+ enoff_sec->dofs_size) {
dtrace_dof_error(dof, "invalid is-enabled "
"offset");
return (-1);
@@ -16366,13 +16373,17 @@
if (probe->dofpr_argidx + probe->dofpr_xargc <
probe->dofpr_argidx ||
(probe->dofpr_argidx + probe->dofpr_xargc) *
- arg_sec->dofs_entsize > arg_sec->dofs_size) {
+ (uint64_t) arg_sec->dofs_entsize > arg_sec->dofs_size) {
dtrace_dof_error(dof, "invalid args");
return (-1);
}
typeidx = probe->dofpr_nargv;
- typestr = strtab + probe->dofpr_nargv;
+ if (typeidx >= str_sec->dofs_size) {
+ dtrace_dof_error(dof, "bad native argument type");
+ return (-1);
+ }
+ typestr = strtab + typeidx;
for (k = 0; k < probe->dofpr_nargc; k++) {
if (typeidx >= str_sec->dofs_size) {
dtrace_dof_error(dof, "bad "
@@ -16391,9 +16402,14 @@
}
typeidx = probe->dofpr_xargv;
- typestr = strtab + probe->dofpr_xargv;
+ if (typeidx >= str_sec->dofs_size) {
+ dtrace_dof_error(dof, "bad native argument type");
+ return (-1);
+ }
+ typestr = strtab + typeidx;
for (k = 0; k < probe->dofpr_xargc; k++) {
- if (arg[probe->dofpr_argidx + k] > probe->dofpr_nargc) {
+ if (arg[probe->dofpr_argidx + k] >=
+ probe->dofpr_nargc) {
dtrace_dof_error(dof, "bad "
"native argument index");
return (-1);

File Metadata

Mime Type
text/plain
Expires
Sun, Jul 12, 6:19 PM (10 h, 34 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
34577723
Default Alt Text
D57979.id181354.diff (2 KB)

Event Timeline