Re: [stable] [PATCH 5/6] drivers/video/ps3fb: fix memset size error
On Mon, Nov 05, 2007 at 02:00:45PM +0100, Geert Uytterhoeven wrote: > On Mon, 5 Nov 2007, Geert Uytterhoeven wrote: > > On Mon, 5 Nov 2007, Li Zefan wrote: > > > The size passing to memset is wrong. > > > > > > Signed-off-by Li Zefan <[EMAIL PROTECTED]> > > > > Acked-by: Geert Uytterhoeven <[EMAIL PROTECTED]> > > As this leaks 24 bytes of uninitialized stack to user space, probably this > should be applied to -stable, too? I'll be glad to apply it, once it goes to Linus's tree. Can you all resend this to [EMAIL PROTECTED] when it goes into his tree? thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [stable] [PATCH 5/6] drivers/video/ps3fb: fix memset size error
On Mon, Nov 05, 2007 at 02:00:45PM +0100, Geert Uytterhoeven wrote: On Mon, 5 Nov 2007, Geert Uytterhoeven wrote: On Mon, 5 Nov 2007, Li Zefan wrote: The size passing to memset is wrong. Signed-off-by Li Zefan [EMAIL PROTECTED] Acked-by: Geert Uytterhoeven [EMAIL PROTECTED] As this leaks 24 bytes of uninitialized stack to user space, probably this should be applied to -stable, too? I'll be glad to apply it, once it goes to Linus's tree. Can you all resend this to [EMAIL PROTECTED] when it goes into his tree? thanks, greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH 5/6] drivers/video/ps3fb: fix memset size error
On Mon, 5 Nov 2007, Geert Uytterhoeven wrote: > On Mon, 5 Nov 2007, Li Zefan wrote: > > The size passing to memset is wrong. > > > > Signed-off-by Li Zefan <[EMAIL PROTECTED]> > > Acked-by: Geert Uytterhoeven <[EMAIL PROTECTED]> As this leaks 24 bytes of uninitialized stack to user space, probably this should be applied to -stable, too? > > --- > > drivers/video/ps3fb.c |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c > > index b3463dd..75836aa 100644 > > --- a/drivers/video/ps3fb.c > > +++ b/drivers/video/ps3fb.c > > @@ -727,7 +727,7 @@ static int ps3fb_blank(int blank, struct fb_info *info) > > > > static int ps3fb_get_vblank(struct fb_vblank *vblank) > > { > > - memset(vblank, 0, sizeof()); > > + memset(vblank, 0, sizeof(*vblank)); > > vblank->flags = FB_VBLANK_HAVE_VSYNC; > > return 0; > > } With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: [EMAIL PROTECTED] Internet: http://www.sony-europe.com/ Sony Network and Software Technology Center Europe A division of Sony Service Centre (Europe) N.V. Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium VAT BE 0413.825.160 · RPR Brussels Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
Re: [PATCH 5/6] drivers/video/ps3fb: fix memset size error
On Mon, 5 Nov 2007, Li Zefan wrote: > The size passing to memset is wrong. > > Signed-off-by Li Zefan <[EMAIL PROTECTED]> Acked-by: Geert Uytterhoeven <[EMAIL PROTECTED]> > --- > drivers/video/ps3fb.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c > index b3463dd..75836aa 100644 > --- a/drivers/video/ps3fb.c > +++ b/drivers/video/ps3fb.c > @@ -727,7 +727,7 @@ static int ps3fb_blank(int blank, struct fb_info *info) > > static int ps3fb_get_vblank(struct fb_vblank *vblank) > { > - memset(vblank, 0, sizeof()); > + memset(vblank, 0, sizeof(*vblank)); > vblank->flags = FB_VBLANK_HAVE_VSYNC; > return 0; > } With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: [EMAIL PROTECTED] Internet: http://www.sony-europe.com/ Sony Network and Software Technology Center Europe A division of Sony Service Centre (Europe) N.V. Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium VAT BE 0413.825.160 · RPR Brussels Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
Re: [PATCH 5/6] drivers/video/ps3fb: fix memset size error
On Mon, 5 Nov 2007, Li Zefan wrote: The size passing to memset is wrong. Signed-off-by Li Zefan [EMAIL PROTECTED] Acked-by: Geert Uytterhoeven [EMAIL PROTECTED] --- drivers/video/ps3fb.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c index b3463dd..75836aa 100644 --- a/drivers/video/ps3fb.c +++ b/drivers/video/ps3fb.c @@ -727,7 +727,7 @@ static int ps3fb_blank(int blank, struct fb_info *info) static int ps3fb_get_vblank(struct fb_vblank *vblank) { - memset(vblank, 0, sizeof(vblank)); + memset(vblank, 0, sizeof(*vblank)); vblank-flags = FB_VBLANK_HAVE_VSYNC; return 0; } With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: [EMAIL PROTECTED] Internet: http://www.sony-europe.com/ Sony Network and Software Technology Center Europe A division of Sony Service Centre (Europe) N.V. Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium VAT BE 0413.825.160 · RPR Brussels Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
Re: [PATCH 5/6] drivers/video/ps3fb: fix memset size error
On Mon, 5 Nov 2007, Geert Uytterhoeven wrote: On Mon, 5 Nov 2007, Li Zefan wrote: The size passing to memset is wrong. Signed-off-by Li Zefan [EMAIL PROTECTED] Acked-by: Geert Uytterhoeven [EMAIL PROTECTED] As this leaks 24 bytes of uninitialized stack to user space, probably this should be applied to -stable, too? --- drivers/video/ps3fb.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c index b3463dd..75836aa 100644 --- a/drivers/video/ps3fb.c +++ b/drivers/video/ps3fb.c @@ -727,7 +727,7 @@ static int ps3fb_blank(int blank, struct fb_info *info) static int ps3fb_get_vblank(struct fb_vblank *vblank) { - memset(vblank, 0, sizeof(vblank)); + memset(vblank, 0, sizeof(*vblank)); vblank-flags = FB_VBLANK_HAVE_VSYNC; return 0; } With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: [EMAIL PROTECTED] Internet: http://www.sony-europe.com/ Sony Network and Software Technology Center Europe A division of Sony Service Centre (Europe) N.V. Registered office: Technologielaan 7 · B-1840 Londerzeel · Belgium VAT BE 0413.825.160 · RPR Brussels Fortis Bank Zaventem · Swift GEBABEBB08A · IBAN BE39001382358619
[PATCH 5/6] drivers/video/ps3fb: fix memset size error
The size passing to memset is wrong. Signed-off-by Li Zefan <[EMAIL PROTECTED]> --- drivers/video/ps3fb.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c index b3463dd..75836aa 100644 --- a/drivers/video/ps3fb.c +++ b/drivers/video/ps3fb.c @@ -727,7 +727,7 @@ static int ps3fb_blank(int blank, struct fb_info *info) static int ps3fb_get_vblank(struct fb_vblank *vblank) { - memset(vblank, 0, sizeof()); + memset(vblank, 0, sizeof(*vblank)); vblank->flags = FB_VBLANK_HAVE_VSYNC; return 0; } -- 1.5.3.rc7 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[PATCH 5/6] drivers/video/ps3fb: fix memset size error
The size passing to memset is wrong. Signed-off-by Li Zefan [EMAIL PROTECTED] --- drivers/video/ps3fb.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/video/ps3fb.c b/drivers/video/ps3fb.c index b3463dd..75836aa 100644 --- a/drivers/video/ps3fb.c +++ b/drivers/video/ps3fb.c @@ -727,7 +727,7 @@ static int ps3fb_blank(int blank, struct fb_info *info) static int ps3fb_get_vblank(struct fb_vblank *vblank) { - memset(vblank, 0, sizeof(vblank)); + memset(vblank, 0, sizeof(*vblank)); vblank-flags = FB_VBLANK_HAVE_VSYNC; return 0; } -- 1.5.3.rc7 - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/