On Fri, Jul 14, 2017 at 01:56:16PM +0100, Mel Gorman wrote:
> >     if (!write) {
> > -           int len = sizeof("Default");
> > -           if (copy_to_user(buffer, "Default", len))
> > +           int len = sizeof("Node");
> > +           if (copy_to_user(buffer, "Node", len))
> >                     return -EFAULT;
> 
> Ok for the name. But what's with using sizeof?

Bah, sizeof static compile-time string versus char *. Never mind.

-- 
Mel Gorman
SUSE Labs

Reply via email to