Messages in this thread | | | Date | Wed, 30 Apr 2003 22:45:55 -0700 (PDT) | From | Junfeng Yang <> | Subject | Re: [CHECKER] 5 potential user-pointer errors that allow arbitrary reads from kernel |
| |
Also, please note the word "Callstack" in the error reports doesn't really mean an actual call here. It means "functions are linked together if they are assigned to the same structure field".
for example, the following error report is catched in this way:
net/core/pktgen.c:proc_write treats its second arguemtn as tainted. we take it as a good code example, and require that all the other functions that are assigned to proc_dir_entry.write_proc must treat its second argument as tainted.
sorry for the confusions.
> --------------------------------------------------------- > [BUG] proc_dir_entry.write_proc can take tainted inputs > > /home/junfeng/linux-2.5.63/drivers/usb/media/vicam.c:1117:vicam_write_proc_gain: > ERROR:TAINTED:1117:1117: passing tainted ptr 'buffer' to simple_strtoul > [Callstack: > /home/junfeng/linux-2.5.63/net/core/pktgen.c:991:vicam_write_proc_gain((tainted > 1))] > > static int vicam_write_proc_gain(struct file *file, const char *buffer, > unsigned long count, void *data) > { > struct vicam_camera *cam = (struct vicam_camera *)data; > > > Error ---> > cam->gain = simple_strtoul(buffer, NULL, 10); > > return count; > }
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |