Fix compilation warnings to error, mostly use-after-free and maybe un…

…initialized.
micmacIGN · May 16, 2023 · acf9943 · acf9943
1 parent 43022f9
commit acf9943
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 3 deletions.
diff --git a/include/ext_stl/fifo.h b/include/ext_stl/fifo.h
@@ -197,6 +197,9 @@ template <class Type> class ElFifo
 
         void incr_capa()
         {
+            if (_capa == 0)
+                _capa = 1;
+
            Type * NewTab = new Type [2*_capa];
            for (INT kk=0; kk<_nb ; kk++)
                 NewTab[kk] = _tab[(kk+_begin)%_capa];

diff --git a/src/TpMMPD/DeformAnalyse.cpp b/src/TpMMPD/DeformAnalyse.cpp
@@ -76,7 +76,7 @@ int DeformAnalyse_main (int argc, char ** argv)
 
         // 3. get the vector PerResidu
         ifstream aPR((aDir + aOut1).c_str());
-        char *aPerR;
+        char *aPerR{nullptr};
         if(aPR)
         {
             std::string aLine;

diff --git a/src/TpMMPD/TiePByMesh/TaskCorrel/cAppliTaskCorrel.cpp b/src/TpMMPD/TiePByMesh/TaskCorrel/cAppliTaskCorrel.cpp
@@ -116,7 +116,7 @@ void cAppliTaskCorrel::ReadXMLMesurePts(string aGCPMesureXML, vector<cImgForTiep
            std::list<cOneMesureAF1I> & aMes = iT1->OneMesureAF1I();
            string aNameIm = iT1->NameIm();
            cout<<endl<<" + Img : "<<aNameIm<<endl;
-           cImgForTiepTri* aImg;
+           cImgForTiepTri* aImg{};
            for (uint akIm=0; akIm<mVImgs.size(); akIm++)
            {
                if (aNameIm == mVImgs[akIm]->Name())

diff --git a/src/uti_phgrm/GraphCut/MaxFlow/graph.cpp b/src/uti_phgrm/GraphCut/MaxFlow/graph.cpp
@@ -58,6 +58,7 @@ template <typename captype, typename tcaptype, typename flowtype>
 	flow = 0;
 }
 
+/*
 template <typename captype, typename tcaptype, typename flowtype> 
 	void Graph<captype,tcaptype,flowtype>::reallocate_nodes(int num)
 {
@@ -86,7 +87,9 @@ template <typename captype, typename tcaptype, typename flowtype>
 		}
 	}
 }
+*/
 
+/*
 template <typename captype, typename tcaptype, typename flowtype> 
 	void Graph<captype,tcaptype,flowtype>::reallocate_arcs()
 {
@@ -117,5 +120,6 @@ template <typename captype, typename tcaptype, typename flowtype>
 		}
 	}
 }
+*/
 
 #include "instances.inc"
diff --git a/src/util/regex.cpp b/src/util/regex.cpp
@@ -106,7 +106,7 @@ cElRegex::cElRegex(const std::string & aNameExprIn,int aNbMatchMax,int aCFlag,bo
 
    if (! IsOk())
       return;
-   regmatch_t aMatch;
+   regmatch_t aMatch{};
    mVMatch.reserve(aNbMatchMax);
    for (int aK=0; aK<aNbMatchMax ; aK++)
        mVMatch.push_back(aMatch);