[patch 1/4] Add function get_bootparam

Dave Young dyoung at redhat.com
Sun Oct 27 22:30:32 EDT 2013


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



More information about the kexec mailing list