Re: [PATCH] Permit big console scrolls

[Geert Uytterhoeven]

On Sat, 21 Jun 2003, Linux Kernel Mailing List wrote:
> ChangeSet 1.1381, 2003/06/20 22:14:31-07:00, [e-mail]
> 
> 	[PATCH] Permit big console scrolls
> 	
> 	From: Samuel Thibault <[e-mail]>
> 	
> 	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 -- [e-mail]

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