On 10/28/13 at 09:13am, Dave Young wrote:
> > @@ -447,7 +446,7 @@ void setup_subarch(struct
x86_linux_para
> > if (!debugfs_mnt)
> > return;
> > snprintf(filename, PATH_MAX, "%s/%s", debugfs_mnt,
"boot_params/data");
> > - filename[PATH_MAX-1] = 0;
> > + filename[PATH_MAX - 1] = 0;
> > free(debugfs_mnt);
>
> This change appears to be unrelated to the rest of the patch.
>
Will remove the change from the patch.
Relooking it, with this patch the line "filename[PATH_MAX - 1] = 0;"
is in the share function get_bootparam, it's not in setup_subarch
any more so I think it's ok, what do you think?
--
Thanks
Dave