Messages in this thread | | | Date | Mon, 01 Sep 2014 14:54:30 +0100 | From | Rob Jones <> | Subject | Re: [PATCH] fs: replace int param with size_t for seq_open_private() |
| |
I don't agree. Ultimately this parameter ends up as a parameter to kmalloc where it is expected to be a size_t.
On 01/09/14 14:43, Alexey Dobriyan wrote: >> void *__seq_open_private(struct file *f, const struct seq_operations *ops, >> - int psize) >> + size_t psize) > > It should be "unsigned int" at most. > As almost all in-kernel lengths. > > Alexey > >
-- Rob Jones Codethink Ltd mailto:rob.jones@codethink.co.uk tel:+44 161 236 5575
| |