[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2375c9f90812051716w6974a18asd4c34cb47bd64aa4@mail.gmail.com>
Date: Sat, 6 Dec 2008 01:16:34 +0000
From: "Américo Wang" <xiyou.wangcong@...il.com>
To: "Andrew Morton" <akpm@...ux-foundation.org>
Cc: balbir@...ux.vnet.ibm.com, penberg@...helsinki.fi,
jdike@...toit.com, linux-kernel@...r.kernel.org,
user-mode-linux-devel@...ts.sourceforge.net, stable@...nel.org
Subject: Re: [PATCH][UML] Boot broken due to buffer overrun
On Tue, Dec 2, 2008 at 8:17 PM, Andrew Morton <akpm@...ux-foundation.org> wrote:
> On Sat, 29 Nov 2008 17:50:39 +0530
> Balbir Singh <balbir@...ux.vnet.ibm.com> wrote:
>
>> * Pekka Enberg <penberg@...helsinki.fi> [2008-11-29 12:21:46]:
>>
>> > Hi Balbir,
>> >
>> > On Sat, Nov 29, 2008 at 12:08 PM, Balbir Singh
>> > <balbir@...ux.vnet.ibm.com> wrote:
>> > > mconsole_init() passed 256 bytes as length in os_create_unix_socket, while
>> > > the sizeof UNIX_PATH_MAX is 108. This patch fixes that problem and avoids
>> > > a big overrun bug reported on UML bootup.
>> >
>> > Maybe mention that ->sun_path in struct sockaddr_un is UNIX_PATH_MAX
>> > long which causes the problem?
>> >
>>
>> Yes, the changelog can definitely add that to the changelog, I'll
>> resend the patch if Jeff asks for it.
>
> Jeff's been quiet lately - I'll merge this one.
Yes, indeed.
>
> I also tagged it for -stable, subject to Jeff's OK. I'm wondering why
> this wasn't observed in earlier kernels?
Maybe because mconsole is not often used for uml? :)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists