lkml.org 
[lkml]   [2003]   [Jun]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: Misc 2.5 Fixes: cp-user-zoran
From
On Tuesday, Jun 10, 2003, at 05:22 US/Central, Dipankar Sarma wrote:
>
> diff -puN drivers/media/video/zr36120.c~cp-user-zoran
> drivers/media/video/zr36120.c
> ---
> linux-2.5.70-ds/drivers/media/video/zr36120.c~cp-user-zoran 2003-06-08
> 04:10:38.000000000 +0530
> +++ linux-2.5.70-ds-dipankar/drivers/media/video/zr36120.c 2003-06-08
> 04:10:38.000000000 +0530
> @@ -1693,12 +1693,12 @@ long vbi_read(struct video_device* dev,
> for (x=0; optr+1<eptr && x<-done->w; x++)
> {
> unsigned char a = iptr[x*2];
> - *optr++ = a;
> - *optr++ = a;
> + __put_user(a, optr++);
> + __put_user(a, optr++);
> }
> /* and clear the rest of the line */
> for (x*=2; optr<eptr && x<done->bpl; x++)
> - *optr++ = 0;
> + __put_user(0, optr++);
> /* next line */
> iptr += done->bpl;
> }
> @@ -1715,10 +1715,10 @@ long vbi_read(struct video_device* dev,
> {
> /* copy to doubled data to userland */
> for (x=0; optr<eptr && x<-done->w; x++)
> - *optr++ = iptr[x*2];
> + __put_user(iptr[x*2], optr++);
> /* and clear the rest of the line */
> for (;optr<eptr && x<done->bpl; x++)
> - *optr++ = 0;
> + __put_user(0, optr++);
> /* next line */
> iptr += done->bpl;
> }
> @@ -1727,7 +1727,7 @@ long vbi_read(struct video_device* dev,
> /* API compliance:
> * place the framenumber (half fieldnr) in the last long
> */
> - ((ulong*)eptr)[-1] = done->fieldnr/2;
> + __put_user(done->fieldnr/2, ((ulong*)eptr)-1);
> }
>
> /* keep the engine running */

It's funny, I did the exact same thing for the version currently in
bk... but I just realized that __put_user still returns an error code,
so all those calls should be checked.

--
Hollis Blanchard
IBM Linux Technology Center

-
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 13:36    [W:0.146 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site