Staging: bcm: signedness bug in StoreSFParam()

[mv-sheeva.git] / drivers / staging / bcm / CmHost.c

diff --git a/drivers/staging/bcm/CmHost.c b/drivers/staging/bcm/CmHost.c
index a685cadb778d0606d7afad04eac9970b68d0b124..5ac45820d56464b820c2c736b79720a19a0dcb5a 100644 (file)
--- a/drivers/staging/bcm/CmHost.c
+++ b/drivers/staging/bcm/CmHost.c
@@ -1649,17 +1649,17 @@ static inline ULONG RestoreSFParam(PMINI_ADAPTER Adapter, ULONG ulAddrSFParamSet
 static ULONG StoreSFParam(PMINI_ADAPTER Adapter,PUCHAR pucSrcBuffer,ULONG  ulAddrSFParamSet)
 {
     UINT       nBytesToWrite = sizeof(stServiceFlowParamSI);
-       UINT    uiRetVal =0;
+       int ret = 0;
 
        if(ulAddrSFParamSet == 0 || NULL == pucSrcBuffer)
        {
                return 0;
        }
 
-       uiRetVal = wrm(Adapter,ulAddrSFParamSet,(PUCHAR)pucSrcBuffer, nBytesToWrite);
-       if(uiRetVal < 0) {
+       ret = wrm(Adapter, ulAddrSFParamSet, (u8 *)pucSrcBuffer, nBytesToWrite);
+       if (ret < 0) {
                BCM_DEBUG_PRINT( Adapter,DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL,  "%s:%d WRM failed",__FUNCTION__, __LINE__);
-               return uiRetVal;
+               return ret;
        }
        return 1;
 }