Remove never read variable assignments (#95)

In all 3 cases the variables are written again a few lines later. The only purpose could be if xmlSecBufferGetSize() would have a side effect, which is not the case.
lsh123 · Apr 1, 2017 · fe256f6 · fe256f6
1 parent 6c44213
commit fe256f6
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/src/nss/x509.c b/src/nss/x509.c
@@ -900,7 +900,6 @@ xmlSecNssX509DataNodeRead(xmlSecKeyDataPtr data, xmlNodePtr node, xmlSecKeyInfoC
         cur != NULL;
         cur = xmlSecGetNextElementNode(cur->next)) {
 
-        ret = 0;
         if(xmlSecCheckNodeName(cur, xmlSecNodeX509Certificate, xmlSecDSigNs)) {
             ret = xmlSecNssX509CertificateNodeRead(data, cur, keyInfoCtx);
             if(ret < 0) {

diff --git a/src/transforms.c b/src/transforms.c
@@ -1915,8 +1915,6 @@ xmlSecTransformDefaultPushBin(xmlSecTransformPtr transform, const xmlSecByte* da
         }
 
         /* process data */
-        inSize = xmlSecBufferGetSize(&(transform->inBuf));
-        outSize = xmlSecBufferGetSize(&(transform->outBuf));
         finalData = (((dataSize == 0) && (final != 0)) ? 1 : 0);
         ret = xmlSecTransformExecute(transform, finalData, transformCtx);
         if(ret < 0) {