]> Gitweb @ Texas Instruments - Open Source Git Repositories - git.TI.com/gitweb - keystone-rtos/netapi.git/commitdiff
Resolve coverity warning in pktio.c
authorTinku Mannan <tmannan@ti.com>
Fri, 14 Nov 2014 19:59:36 +0000 (14:59 -0500)
committerTinku Mannan <tmannan@ti.com>
Fri, 14 Nov 2014 19:59:36 +0000 (14:59 -0500)
ti/runtime/netapi/src/pktio.c

index cbe802e79bb036d5b03565923a010e8bc975f85d..f0c1ae06844b4cdfa5ffa6c36d7e0d68701acf46 100755 (executable)
@@ -391,6 +391,7 @@ static int netapip_pktioSendPort(struct PKTIO_HANDLE_tag * pp,
     Qmss_queuePushDescSizeRaw(p->tx_psCmdInfo.txQueue,
                                   pPloadDesc,
                                   NWAL_DESC_SIZE);
+    *err = NETAPI_ERR_OK;
     return 1;
 }
 
@@ -407,6 +408,7 @@ static int netapip_pktioSendIpc(struct PKTIO_HANDLE_tag * pp,
     PKTIO_HANDLE_T *p=(PKTIO_HANDLE_T*) pp;
     err=0;
     Qmss_queuePushDesc (p->q, (void*)pkt);
+    *err = NETAPI_ERR_OK;
     return 1;
 }
 /********************************************************************
@@ -422,7 +424,6 @@ static int netapip_pktioSendNwal(struct PKTIO_HANDLE_tag * pp,
     PKTIO_HANDLE_T *p=(PKTIO_HANDLE_T*) pp;
     nwalTxPktInfo_t * pPktInfo=m->u.tx_meta;
     nwal_RetValue res;
-    *err=0;
     pPktInfo->pPkt = pkt;
     res=nwal_send(p->nwalInstanceHandle, m->sa_handle,pPktInfo);
     if (res != nwal_OK)
@@ -430,6 +431,7 @@ static int netapip_pktioSendNwal(struct PKTIO_HANDLE_tag * pp,
         *err = NETAPI_ERR_NWAL_TX_ERR -res;
        return -1;
     }
+    *err=NETAPI_ERR_OK;
     return 1;
 }
 
@@ -758,7 +760,6 @@ PKTIO_HANDLE_T * netapi_pktioCreate(NETAPI_T n,
                                 &isAllocated);
     if (p->q < 0)
     {
-        netapi_Log("netapi_pktioCreate:  queueOpen failed\n");
         p->inuse=0;
         *err= NETAPI_ERR_QLLD;  ///queue lld error
         return NULL;
@@ -905,7 +906,6 @@ PKTIO_HANDLE_T * netapi_pktioOpen(NETAPI_T  n,
 
         if (!p_qnum ) 
         {
-            netapi_Log("netapi_pktioOpen: can't find %s\n",name);
             p->inuse=0;
             *err= NETAPI_ERR_NOTFOUND;  ///queue lld error
             return NULL;
@@ -917,13 +917,11 @@ PKTIO_HANDLE_T * netapi_pktioOpen(NETAPI_T  n,
                            &isAllocated);
         if (p->q == (Qmss_QueueHnd) NULL)
         {
-            netapi_Log("netapi_pktioCreate:  queueOpen failed\n");
             p->inuse=0;
             *err= NETAPI_ERR_QLLD;  ///queue lld error
             return NULL;
         }
         p->qInfo = Qmss_getQueueNumber(p->q);
-        netapi_Log("netapi_pktioOpen: queueMgr %d,  queueNum; %d\n", p->qInfo.qMgr, p->qInfo.qNum);
         if (p_cfg->flags2 & PKTIO_PKT)
         {
            p->use_nwal = PKTIO_4_ADJ_NWAL;  //additonal RX q for nwal
@@ -954,7 +952,6 @@ PKTIO_HANDLE_T * netapi_pktioOpen(NETAPI_T  n,
            strlen(name)<PKTIO_MAX_NAME ?
                    strlen(name):PKTIO_MAX_NAME);
 
-        netapi_Log("netapi_pktioOpen: returning with sucess for name %s\n", p->name);
     return p;
 }
 
@@ -1049,7 +1046,6 @@ void netapi_pktioControl(PKTIO_HANDLE_T * p,
                 break;
             case(PKTIO_DIVERT):
             default:
-                netapi_Log("netapi_pktioControl: pktio_control op %d not implemented\n",p_control->op);
                 *err= NETAPI_ERR_NOT_IMPLEMENTED;
                 break;
         }