On 01/31/11 04:01 AM, Erkki Seppälä wrote:
> Pointer "pBuf" returned from "fgets(buf, 256, stream)" is never used
> 
> Reviewed-by: Ander Conselvan de Oliveira 
> <[email protected]>
> Signed-off-by: Erkki Seppälä <[email protected]>
> ---
>  src/xcms/cmsColNm.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/xcms/cmsColNm.c b/src/xcms/cmsColNm.c
> index 4164370..c7ad4fd 100644
> --- a/src/xcms/cmsColNm.c
> +++ b/src/xcms/cmsColNm.c
> @@ -567,7 +567,7 @@ stringSectionSize(
>       return(XcmsFailure);
>      }
>  
> -    while((pBuf = fgets(buf, XCMSDB_MAXLINELEN, stream)) != NULL) {
> +    while((fgets(buf, XCMSDB_MAXLINELEN, stream)) != NULL) {
>       if ((sscanf(buf, "%s", token)) && (strcmp(token, END_TOKEN) == 0)) {
>           break;
>       }

Reviewed-by: Alan Coopersmith <[email protected]>

-- 
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to