lkml.org 
[lkml]   [2003]   [Jun]   [22]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateSun, 22 Jun 2003 11:04:33 +0200 (MEST)
FromGeert Uytterhoeven <>
SubjectRe: [PATCH] Permit big console scrolls
On Sat, 21 Jun 2003, Linux Kernel Mailing List wrote:
> ChangeSet 1.1381, 2003/06/20 22:14:31-07:00, akpm@digeo.com
> 
> 	[PATCH] Permit big console scrolls
> 
> 	From: Samuel Thibault <Samuel.Thibault@ens-lyon.fr>
> 
> 	Changes the new console scrolling ioctl to permit distances greater than
> 	+127/-128.
> 
> 
> # This patch includes the following deltas:
> #	           ChangeSet	1.1380  -> 1.1381 
> #	   drivers/char/vt.c	1.52    -> 1.53   
> #
> 
>  vt.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletion(-)
> 
> 
> diff -Nru a/drivers/char/vt.c b/drivers/char/vt.c
> --- a/drivers/char/vt.c	Fri Jun 20 23:21:41 2003
> +++ b/drivers/char/vt.c	Fri Jun 20 23:21:41 2003
> @@ -75,6 +75,7 @@
>   */
> 
>  #include <linux/module.h>
> +#include <linux/types.h>
>  #include <linux/sched.h>
>  #include <linux/tty.h>
>  #include <linux/tty_flip.h>
> @@ -2279,7 +2280,7 @@
>  			ret = fg_console;
>  			break;
>  		case TIOCL_SCROLLCONSOLE:
> -			if (get_user(lines, (char *)arg+1)) {
						    ^^^^^
> +			if (get_user(lines, (s32 *)((char *)arg+4))) {
							    ^^^^^
>  				ret = -EFAULT;
>  			} else {
>  				scrollfront(lines);

Why was the `arg+1' changed to `arg+4'? Do we really want to skip 12 bytes?

Gr{oetje,eeting}s,
						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:36    [from the cache]
©2003-2008