Attachment 10963 Details for Bug 11812 – adjusts the checks...

[patch] adjusts the checks...

0009-Fixed-return-value-handling-of-fwrite.patch (text/plain), 2.37 KB, created by Roland "Test-tools" Bär on 2007-08-02 06:43:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Roland "Test-tools" Bär

Created: 2007-08-02 06:43:56 UTC

Size: 2.37 KB

patch

obsolete

>From deff87e4029be072db76ebbb807efa1504f94170 Mon Sep 17 00:00:00 2001
>From: Roland Baer <roland@Vista.(none)>
>Date: Thu, 2 Aug 2007 16:35:55 +0300
>Subject: [PATCH] Fixed return value handling of fwrite()
>
>---
> src/mesa/drivers/dri/i965/brw_aub.c |   16 ++++++++--------
> 1 files changed, 8 insertions(+), 8 deletions(-)
>
>diff --git a/src/mesa/drivers/dri/i965/brw_aub.c b/src/mesa/drivers/dri/i965/brw_aub.c
>index f851a5b..c549f7a 100644
>--- a/src/mesa/drivers/dri/i965/brw_aub.c
>+++ b/src/mesa/drivers/dri/i965/brw_aub.c
>@@ -80,7 +80,7 @@ static void init_aubfile( FILE *aub_file )
>    fh.hour = (nr>>16) & 0xff;
>    fh.comment_length = 0x0;   
> 
>-   if (fwrite(&fh, sizeof(fh), 1, aub_file) < 0) 
>+   if (fwrite(&fh, sizeof(fh), 1, aub_file) < 1) 
>       FAIL;
>          
>    /* Setup the GTT starting at main memory address zero (!):
>@@ -96,12 +96,12 @@ static void init_aubfile( FILE *aub_file )
>    bh.address = PGETBL_CTL;
>    bh.length = 0x4;
> 
>-   if (fwrite(&bh, sizeof(bh), 1, aub_file) < 0) 
>+   if (fwrite(&bh, sizeof(bh), 1, aub_file) < 1) 
>       FAIL;
> 
>    data = 0x0 | PGETBL_ENABLED;
> 
>-   if (fwrite(&data, sizeof(data), 1, aub_file) < 0) 
>+   if (fwrite(&data, sizeof(data), 1, aub_file) < 1) 
>       FAIL;
> }
> 
>@@ -128,7 +128,7 @@ static void init_aub_gtt( struct brw_context *brw,
>    bh.address =  start_offset / 4096 * 4;
>    bh.length = size / 4096 * 4;
> 
>-   if (fwrite(&bh, sizeof(bh), 1, aub_file) < 0) 
>+   if (fwrite(&bh, sizeof(bh), 1, aub_file) < 1) 
>       FAIL;
> 
>    for (i = 0; i < size / 4096; i++) {
>@@ -136,7 +136,7 @@ static void init_aub_gtt( struct brw_context *brw,
> 
>       brw->next_free_page += 4096;
> 
>-      if (fwrite(&data, sizeof(data), 1, aub_file) < 0) 
>+      if (fwrite(&data, sizeof(data), 1, aub_file) < 1) 
> 	 FAIL;
>    }
> 
>@@ -149,10 +149,10 @@ static void write_block_header( FILE *aub_file,
> {
>    sz = (sz + 3) & ~3;
> 
>-   if (fwrite(bh, sizeof(*bh), 1, aub_file) < 0) 
>+   if (fwrite(bh, sizeof(*bh), 1, aub_file) < 1) 
>       FAIL;
> 
>-   if (fwrite(data, sz, 1, aub_file) < 0) 
>+   if (fwrite(data, sz, 1, aub_file) < 1) 
>       FAIL;
> 
>    fflush(aub_file);
>@@ -162,7 +162,7 @@ static void write_block_header( FILE *aub_file,
> static void write_dump_bmp( FILE *aub_file,
> 			    struct aub_dump_bmp *db )
> {
>-   if (fwrite(db, sizeof(*db), 1, aub_file) < 0) 
>+   if (fwrite(db, sizeof(*db), 1, aub_file) < 1) 
>       FAIL;
> 
>    fflush(aub_file);
>-- 
>1.5.2.3
>

Actions: View | Splinter Review

Attachments on bug 11812: 10963

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.