[PATCH] vmcore-dmesg: Do not write beyond end of allocated buffer
Simon Horman
horms at verge.net.au
Mon Jul 23 19:08:48 EDT 2012
On Mon, Jul 23, 2012 at 10:35:12AM -0400, Vivek Goyal wrote:
> On Wed, Jul 18, 2012 at 09:33:51AM -0400, Vivek Goyal wrote:
>
> [..]
> > > > + char temp_buf[1024];
> > > > + bool last_line = false;
> > >
> > > Is there a chance of over-running temp_buf?
> > >
> > > I would be more comfortable if there was a check below to
> > > ensure that len can never be greater than sizeof(temp_buf).
> >
> > Hi Simon,
> >
> > Agreed. Making sure temp_buf does not overflow is a good idea. Here is
> > the udpated patch.
> >
>
> Hi Simon,
>
> Do you have any more concerns with this patch. If not, can you please
> apply it.
Sorry, I missed the updated patch somehow.
I'll apply it ASAP.
More information about the kexec
mailing list