[Digikam-devel] New Defects reported by Coverity Scan for digiKam

Gilles Caulier caulier.gilles at gmail.com
Sat Mar 2 13:30:27 GMT 2013


Hi all.

I setup Coverity SCAN to send automatically a mail to digiKam-devel and
kde-imaging ML when parsing is complete.

The new scan done this morning has included :

- digiKam code from "tableview" currently written by Michael G Hansen (3
minor reports).
- libkface from "opentld" branches currently written by Mahesh Hegde (31
issues from OpenTLD implementation).

All others components are taken from git/master

Best

Gilles Caulier


2013/3/2 <scan-admin at coverity.com>

>
> Hi,
>
> Please find the latest report on new defect(s) introduced to digiKam found
> with Coverity SCAN
>
> Defect(s) Reported-by: Coverity Scan
> Showing 7 of 34 defects
>
> ** CID 989146: Uninitialized scalar field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.cpp: 43
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989146
>
> ** CID 989145: Uninitialized scalar field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/opentld/main/Settings.cpp: 54
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989145
>
> ** CID 989144: Uninitialized pointer field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/VarianceFilter.cpp:
> 43
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989144
>
> ** CID 989143: Uninitialized scalar field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/TLD.cpp: 55
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989143
>
> ** CID 989142: Uninitialized pointer field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.cpp:
> 45
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989142
>
> ** CID 989141: Uninitialized scalar field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/IntegralImage.h:
> 47
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989141
>
> ** CID 989140: Uninitialized pointer field (UNINIT_CTOR)
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/ForegroundDetector.cpp:
> 39
>
> http://scan5.coverity.com:8080//sourcebrowser.htm?projectId=10358#mergedDefectId=989140
>
>
> ________________________________________________________________________
> CID 989146: Uninitialized scalar field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.h: 71 (
> member_decl)
>    68        QString serialisedLeaves;
>    69        QString Name;
>    70
> >>> Class member declaration for "faceid".
>    71        int     faceid;
>    72        int     objHeight;
>    73        int     objWidth;
>    74
>    75        float   minVar;
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.cpp: 43 (
> uninit_member)
>    40
>    41    unitFaceModel::unitFaceModel()
>    42    {
> >>> Non-static class member "faceid" is not initialized in this
> constructor nor in any functions that it calls.
>    43    }
>    44
>    45    unitFaceModel::~unitFaceModel()
>    46    {
>    47    }
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.h: 72 (
> member_decl)
>    69        QString Name;
>    70
>    71        int     faceid;
> >>> Class member declaration for "objHeight".
>    72        int     objHeight;
>    73        int     objWidth;
>    74
>    75        float   minVar;
>    76    };
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.cpp: 43 (
> uninit_member)
>    40
>    41    unitFaceModel::unitFaceModel()
>    42    {
> >>> Non-static class member "objHeight" is not initialized in this
> constructor nor in any functions that it calls.
>    43    }
>    44
>    45    unitFaceModel::~unitFaceModel()
>    46    {
>    47    }
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.h: 73 (
> member_decl)
>    70
>    71        int     faceid;
>    72        int     objHeight;
> >>> Class member declaration for "objWidth".
>    73        int     objWidth;
>    74
>    75        float   minVar;
>    76    };
>    77
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.cpp: 43 (
> uninit_member)
>    40
>    41    unitFaceModel::unitFaceModel()
>    42    {
> >>> Non-static class member "objWidth" is not initialized in this
> constructor nor in any functions that it calls.
>    43    }
>    44
>    45    unitFaceModel::~unitFaceModel()
>    46    {
>    47    }
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.h: 75 (
> member_decl)
>    72        int     objHeight;
>    73        int     objWidth;
>    74
> >>> Class member declaration for "minVar".
>    75        float   minVar;
>    76    };
>    77
>    78    } // namespace libface
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/tldinterface/tldface.cpp: 43 (
> uninit_member)
>    40
>    41    unitFaceModel::unitFaceModel()
>    42    {
> >>> CID 989146: Uninitialized scalar field (UNINIT_CTOR)
> >>> Non-static class member "minVar" is not initialized in this
> constructor nor in any functions that it calls.
>    43    }
>    44
>    45    unitFaceModel::~unitFaceModel()
>    46    {
>    47    }
>
> ________________________________________________________________________
> CID 989145: Uninitialized scalar field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/opentld/main/Settings.h: 49
> ( member_decl)
>    46        bool m_alternating; //!< if set to true, detector is disabled
> while tracker is running.
>    47        bool m_exportModelAfterRun; //!< if set to true, model is
> exported after run.
>    48        int m_trajectory; //!< specifies the number of the last
> frames which are considered by the trajectory; 0 disables the trajectory
> >>> Class member declaration for "m_method".
>    49        int m_method; //!< method of capturing: IMACQ_CAM, IMACQ_IMGS
> or IMACQ_VID
>    50        int m_startFrame; //!< first frame of capturing
>    51        int m_lastFrame; //!< last frame of caputing; 0 means take
> all frames
>    52        int m_minScale; //!< number of scales smaller than initial
> object size
>    53        int m_maxScale; //!< number of scales larger than initial
> object size
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/opentld/main/Settings.cpp:
> 54 ( uninit_member)
>    51        m_proportionalShift(0.1),
>    52        m_initialBoundingBox(vector<int>())
>    53    {
> >>> CID 989145: Uninitialized scalar field (UNINIT_CTOR)
> >>> Non-static class member "m_method" is not initialized in this
> constructor nor in any functions that it calls.
>    54    }
>    55
>    56    Settings::~Settings()
>    57    {
>    58    }
>
> ________________________________________________________________________
> CID 989144: Uninitialized pointer field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/VarianceFilter.h:
> 44 ( member_decl)
>    41
>    42    public:
>    43        bool enabled;
> >>> Class member declaration for "windowOffsets".
>    44        int *windowOffsets;
>    45
>    46        DetectionResult *detectionResult;
>    47
>    48        float minVar;
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/VarianceFilter.cpp:
> 43 ( uninit_member)
>    40        minVar = 0;
>    41        integralImg = NULL;
>    42        integralImg_squared = NULL;
> >>> Non-static class member "windowOffsets" is not initialized in this
> constructor nor in any functions that it calls.
>    43    }
>    44
>    45    VarianceFilter::~VarianceFilter()
>    46    {
>    47        release();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/VarianceFilter.h:
> 46 ( member_decl)
>    43        bool enabled;
>    44        int *windowOffsets;
>    45
> >>> Class member declaration for "detectionResult".
>    46        DetectionResult *detectionResult;
>    47
>    48        float minVar;
>    49
>    50        VarianceFilter();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/VarianceFilter.cpp:
> 43 ( uninit_member)
>    40        minVar = 0;
>    41        integralImg = NULL;
>    42        integralImg_squared = NULL;
> >>> CID 989144: Uninitialized pointer field (UNINIT_CTOR)
> >>> Non-static class member "detectionResult" is not initialized in this
> constructor nor in any functions that it calls.
>    43    }
>    44
>    45    VarianceFilter::~VarianceFilter()
>    46    {
>    47        release();
>
> ________________________________________________________________________
> CID 989143: Uninitialized scalar field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/TLD.h: 62 (
> member_decl)
>    59        cv::Mat currImg;
>    60        cv::Rect *prevBB;
>    61        cv::Rect *currBB;
> >>> Class member declaration for "currConf".
>    62        float currConf;
>    63        bool learning;
>    64
>    65        TLD();
>    66        virtual ~TLD();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/TLD.cpp: 55 (
> uninit_member)
>    52        nnClassifier = detectorCascade->nnClassifier;
>    53
>    54        medianFlowTracker = new MedianFlowTracker();
> >>> CID 989143: Uninitialized scalar field (UNINIT_CTOR)
> >>> Non-static class member "currConf" is not initialized in this
> constructor nor in any functions that it calls.
>    55    }
>    56
>    57    TLD::~TLD()
>    58    {
>    59        storeCurrentData();
>
> ________________________________________________________________________
> CID 989142: Uninitialized pointer field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.h:
> 43 ( member_decl)
>    40    {
>    41        float ncc(float *f1, float *f2);
>    42    public:
> >>> Class member declaration for "enabled".
>    43        bool enabled;
>    44
>    45        int *windows;
>    46        float thetaFP;
>    47        float thetaTP;
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.cpp:
> 45 ( uninit_member)
>    42        truePositives = new vector<NormalizedPatch>();
>    43        falsePositives = new vector<NormalizedPatch>();
>    44
> >>> Non-static class member "enabled" is not initialized in this
> constructor nor in any functions that it calls.
>    45    }
>    46
>    47    NNClassifier::~NNClassifier()
>    48    {
>    49        release();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.h:
> 45 ( member_decl)
>    42    public:
>    43        bool enabled;
>    44
> >>> Class member declaration for "windows".
>    45        int *windows;
>    46        float thetaFP;
>    47        float thetaTP;
>    48        DetectionResult *detectionResult;
>    49        std::vector<NormalizedPatch>* falsePositives;
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.cpp:
> 45 ( uninit_member)
>    42        truePositives = new vector<NormalizedPatch>();
>    43        falsePositives = new vector<NormalizedPatch>();
>    44
> >>> Non-static class member "windows" is not initialized in this
> constructor nor in any functions that it calls.
>    45    }
>    46
>    47    NNClassifier::~NNClassifier()
>    48    {
>    49        release();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.h:
> 48 ( member_decl)
>    45        int *windows;
>    46        float thetaFP;
>    47        float thetaTP;
> >>> Class member declaration for "detectionResult".
>    48        DetectionResult *detectionResult;
>    49        std::vector<NormalizedPatch>* falsePositives;
>    50        std::vector<NormalizedPatch>* truePositives;
>    51
>    52        NNClassifier();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/NNClassifier.cpp:
> 45 ( uninit_member)
>    42        truePositives = new vector<NormalizedPatch>();
>    43        falsePositives = new vector<NormalizedPatch>();
>    44
> >>> CID 989142: Uninitialized pointer field (UNINIT_CTOR)
> >>> Non-static class member "detectionResult" is not initialized in this
> constructor nor in any functions that it calls.
>    45    }
>    46
>    47    NNClassifier::~NNClassifier()
>    48    {
>    49        release();
>
> ________________________________________________________________________
> CID 989141: Uninitialized scalar field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/IntegralImage.h:
> 41 ( member_decl)
>    38    public:
>    39        T *data;  /* Array containg the entries for the integral
> image in row-first manner. Of size width*height. Allocated by
> tldAllocIntImg */
>    40        /* width, height: Dimensions of integral image.*/
> >>> Class member declaration for "width".
>    41        int width;
>    42        int height;
>    43
>    44        IntegralImage(cv::Size size)
>    45        {
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/IntegralImage.h:
> 47 ( uninit_member)
>    44        IntegralImage(cv::Size size)
>    45        {
>    46            data = new T[size.width * size.height];
> >>> Non-static class member "width" is not initialized in this constructor
> nor in any functions that it calls.
>    47        }
>    48
>    49        virtual ~IntegralImage()
>    50        {
>    51            delete[] data;
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/IntegralImage.h:
> 42 ( member_decl)
>    39        T *data;  /* Array containg the entries for the integral
> image in row-first manner. Of size width*height. Allocated by
> tldAllocIntImg */
>    40        /* width, height: Dimensions of integral image.*/
>    41        int width;
> >>> Class member declaration for "height".
>    42        int height;
>    43
>    44        IntegralImage(cv::Size size)
>    45        {
>    46            data = new T[size.width * size.height];
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/IntegralImage.h:
> 47 ( uninit_member)
>    44        IntegralImage(cv::Size size)
>    45        {
>    46            data = new T[size.width * size.height];
> >>> CID 989141: Uninitialized scalar field (UNINIT_CTOR)
> >>> Non-static class member "height" is not initialized in this
> constructor nor in any functions that it calls.
>    47        }
>    48
>    49        virtual ~IntegralImage()
>    50        {
>    51            delete[] data;
>
> ________________________________________________________________________
> CID 989140: Uninitialized pointer field (UNINIT_CTOR)
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/ForegroundDetector.h:
> 45 ( member_decl)
>    42        int fgThreshold;
>    43        int minBlobSize;
>    44        cv::Mat bgImg;
> >>> Class member declaration for "detectionResult".
>    45        DetectionResult *detectionResult;
>    46
>    47        ForegroundDetector();
>    48        virtual ~ForegroundDetector();
>    49        void release();
>
>
> /mnt/devel/GIT/3.x/extra/libkface/OpenTLD/src/libopentld/tld/ForegroundDetector.cpp:
> 39 ( uninit_member)
>    36    {
>    37        fgThreshold = 16;
>    38        minBlobSize = 0;
> >>> CID 989140: Uninitialized pointer field (UNINIT_CTOR)
> >>> Non-static class member "detectionResult" is not initialized in this
> constructor nor in any functions that it calls.
>    39    }
>    40
>    41    ForegroundDetector::~ForegroundDetector()
>    42    {
>    43    }
>
> ________________________________________________________________________
> To view the defects in Coverity Scan visit, http://scan5.coverity.com:8080
>
> If you don't have a username, you can request one by emailing:
> scan-admin at coverity.com
>
> To unsubscribe from the email notification for new defects,
> http://scan.coverity.com/email_unsubscribe.html
> If you are project owner, you can subscribe your team member by accessing
> http://scan.coverity.com/email_subscription.html
> _______________________________________________
> Digikam-devel mailing list
> Digikam-devel at kde.org
> https://mail.kde.org/mailman/listinfo/digikam-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20130302/57c684b0/attachment.html>


More information about the Digikam-devel mailing list