Fixed defects

List of Defects

Error: COMPILER_WARNING (CWE-477): [#def1]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp: scope_hint: In function ‘const float& Imf_3_1::dwaCompressionLevel(const Header&)’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:55:41: warning[-Wdeprecated-declarations]: ‘const Imf_3_1::TypedAttribute<float>& Imf_3_1::dwaCompressionLevelAttribute(const Header&)’ is deprecated: use compression method in ImfHeader
#   55 |         return IMF_NAME_ATTRIBUTE(name) (header).value();           \
#      |                                         ^
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:99:1: note: in expansion of macro ‘IMF_STD_ATTRIBUTE_IMP’
#   99 | IMF_STD_ATTRIBUTE_IMP (dwaCompressionLevel, DwaCompressionLevel, float)
#      | ^~~~~~~~~~~~~~~~~~~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:12: included_from: Included from here.
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:99:24: note: declared here
#   99 | IMF_STD_ATTRIBUTE_IMP (dwaCompressionLevel, DwaCompressionLevel, float)
#      |                        ^~~~~~~~~~~~~~~~~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.h:46:34: note: in definition of macro ‘IMF_NAME_ATTRIBUTE’
#   46 | #define IMF_NAME_ATTRIBUTE(name) name##Attribute
#      |                                  ^~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:99:1: note: in expansion of macro ‘IMF_STD_ATTRIBUTE_IMP’
#   99 | IMF_STD_ATTRIBUTE_IMP (dwaCompressionLevel, DwaCompressionLevel, float)
#      | ^~~~~~~~~~~~~~~~~~~~~
#   53|       name (const Header &header)						 \
#   54|       {									 \
#   55|-> 	return IMF_NAME_ATTRIBUTE(name) (header).value();           \
#   56|       }									 \
#   57|   									 \

Error: COMPILER_WARNING (CWE-477): [#def2]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp: scope_hint: In function ‘float& Imf_3_1::dwaCompressionLevel(Header&)’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:61:41: warning[-Wdeprecated-declarations]: ‘Imf_3_1::TypedAttribute<float>& Imf_3_1::dwaCompressionLevelAttribute(Header&)’ is deprecated: use compression method in ImfHeader
#   61 |         return IMF_NAME_ATTRIBUTE(name) (header).value();           \
#      |                                         ^
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:99:1: note: in expansion of macro ‘IMF_STD_ATTRIBUTE_IMP’
#   99 | IMF_STD_ATTRIBUTE_IMP (dwaCompressionLevel, DwaCompressionLevel, float)
#      | ^~~~~~~~~~~~~~~~~~~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:99:24: note: declared here
#   99 | IMF_STD_ATTRIBUTE_IMP (dwaCompressionLevel, DwaCompressionLevel, float)
#      |                        ^~~~~~~~~~~~~~~~~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.h:46:34: note: in definition of macro ‘IMF_NAME_ATTRIBUTE’
#   46 | #define IMF_NAME_ATTRIBUTE(name) name##Attribute
#      |                                  ^~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXR/ImfStandardAttributes.cpp:99:1: note: in expansion of macro ‘IMF_STD_ATTRIBUTE_IMP’
#   99 | IMF_STD_ATTRIBUTE_IMP (dwaCompressionLevel, DwaCompressionLevel, float)
#      | ^~~~~~~~~~~~~~~~~~~~~
#   59|       name (Header &header)						 \
#   60|       {									 \
#   61|-> 	return IMF_NAME_ATTRIBUTE(name) (header).value();           \
#   62|       }
#   63|   

Error: COMPILER_WARNING (CWE-563): [#def3]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/chunk.c: scope_hint: In function ‘reconstruct_chunk_table’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/chunk.c:460:38: warning[-Wunused-but-set-variable]: variable ‘maxidx’ set but not used
#  460 |     int                              maxidx, found_ci, computed_ci, partnum = 0;
#      |                                      ^~~~~~
#  458|       uint64_t*                        curctable;
#  459|       const struct _internal_exr_part* curpart = NULL;
#  460|->     int                              maxidx, found_ci, computed_ci, partnum = 0;
#  461|   
#  462|       curpart      = ctxt->parts[ctxt->num_parts - 1];

Error: CLANG_WARNING: [#def4]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/chunk.c:485:9: warning[deadcode.DeadStores]: Value stored to 'maxidx' is never read
#  483|   
#  484|           chunk_start = curctable[0];
#  485|->         maxidx      = 0;
#  486|           for (int ci = 1; ci < curpart->chunk_count; ++ci)
#  487|           {

Error: CLANG_WARNING: [#def5]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/chunk.c:490:17: warning[deadcode.DeadStores]: Value stored to 'maxidx' is never read
#  488|               if (curctable[ci] > chunk_start)
#  489|               {
#  490|->                 maxidx      = ci;
#  491|                   chunk_start = curctable[ci];
#  492|               }

Error: COMPILER_WARNING (CWE-457): [#def6]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:58:18: warning[-Wmaybe-uninitialized]: ‘MEM[(unsigned int *)&regs + 8B]’ may be used uninitialized
#   56|           regs[2] = 0;
#   57|   #   else
#   58|->     unsigned int regs[4], osxsave;
#   59|       __get_cpuid (0, &regs[0], &regs[1], &regs[2], &regs[3]);
#   60|       if (regs[0] >= 1)

Error: COMPILER_WARNING (CWE-457): [#def7]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:60:5: warning[-Wmaybe-uninitialized]: ‘regs’ may be used uninitialized
#   60 |     if (regs[0] >= 1)
#      |     ^~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h: scope_hint: In function ‘testCPUIdent(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:58:18: note: ‘regs’ was declared here
#   58 |     unsigned int regs[4], osxsave;
#      |                  ^~~~
#   58|       unsigned int regs[4], osxsave;
#   59|       __get_cpuid (0, &regs[0], &regs[1], &regs[2], &regs[3]);
#   60|->     if (regs[0] >= 1)
#   61|       {
#   62|           __get_cpuid (1, &regs[0], &regs[1], &regs[2], &regs[3]);

Error: COMPILER_WARNING (CWE-457): [#def8]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:60:8: warning[-Wmaybe-uninitialized]: ‘regs’ may be used uninitialized
#   60 |     if (regs[0] >= 1)
#      |        ^
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h: scope_hint: In function ‘DwaCompressor_construct’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:58:18: note: ‘regs’ was declared here
#   58 |     unsigned int regs[4], osxsave;
#      |                  ^~~~
#   58|       unsigned int regs[4], osxsave;
#   59|       __get_cpuid (0, &regs[0], &regs[1], &regs[2], &regs[3]);
#   60|->     if (regs[0] >= 1)
#   61|       {
#   62|           __get_cpuid (1, &regs[0], &regs[1], &regs[2], &regs[3]);

Error: COMPILER_WARNING (CWE-457): [#def9]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:79:37: warning[-Wmaybe-uninitialized]: ‘MEM[(unsigned int *)&regs + 12B]’ may be used uninitialized
#   79 |     *sse2   = (regs[3] & (1 << 26)) ? 1 : 0;
#      |               ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h: scope_hint: In function ‘testCPUIdent(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:58:18: note: ‘MEM[(unsigned int *)&regs + 12B]’ was declared here
#   58 |     unsigned int regs[4], osxsave;
#      |                  ^~~~
#   77|       *f16c   = (regs[2] & (1 << 29)) ? 1 : 0;
#   78|       /* sse2 is in EDX bit 26 */
#   79|->     *sse2   = (regs[3] & (1 << 26)) ? 1 : 0;
#   80|   
#   81|       if (!osxsave)

Error: COMPILER_WARNING (CWE-457): [#def10]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:79:41: warning[-Wmaybe-uninitialized]: ‘MEM[(unsigned int *)&regs + 12B]’ may be used uninitialized
#   79 |     *sse2   = (regs[3] & (1 << 26)) ? 1 : 0;
#      |               ~~~~~~~~~~~~~~~~~~~~~~~~~~^~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h: scope_hint: In function ‘DwaCompressor_construct’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/internal_cpuid.h:58:18: note: ‘MEM[(unsigned int *)&regs + 12B]’ was declared here
#   58 |     unsigned int regs[4], osxsave;
#      |                  ^~~~
#   77|       *f16c   = (regs[2] & (1 << 29)) ? 1 : 0;
#   78|       /* sse2 is in EDX bit 26 */
#   79|->     *sse2   = (regs[3] & (1 << 26)) ? 1 : 0;
#   80|   
#   81|       if (!osxsave)

Error: COMPILER_WARNING (CWE-563): [#def11]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:376:21: warning[-Wunused-but-set-variable]: variable ‘inc0’ set but not used
#  376 |     int             inc0, inc1, inc2;
#      |                     ^~~~
#  374|       uint8_t *       out0, *out1, *out2;
#  375|       int             w, h;
#  376|->     int             inc0, inc1, inc2;
#  377|       int             linc0, linc1, linc2;
#  378|   

Error: COMPILER_WARNING (CWE-563): [#def12]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:376:27: warning[-Wunused-but-set-variable]: variable ‘inc1’ set but not used
#  376 |     int             inc0, inc1, inc2;
#      |                           ^~~~
#  374|       uint8_t *       out0, *out1, *out2;
#  375|       int             w, h;
#  376|->     int             inc0, inc1, inc2;
#  377|       int             linc0, linc1, linc2;
#  378|   

Error: COMPILER_WARNING (CWE-563): [#def13]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c: scope_hint: In function ‘unpack_16bit_3chan_planar’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:376:33: warning[-Wunused-but-set-variable]: variable ‘inc2’ set but not used
#  376 |     int             inc0, inc1, inc2;
#      |                                 ^~~~
#  374|       uint8_t *       out0, *out1, *out2;
#  375|       int             w, h;
#  376|->     int             inc0, inc1, inc2;
#  377|       int             linc0, linc1, linc2;
#  378|   

Error: CLANG_WARNING: [#def14]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:381:5: warning[deadcode.DeadStores]: Value stored to 'inc0' is never read
#  379|       w     = decode->channels[0].width;
#  380|       h     = decode->chunk.height;
#  381|->     inc0  = decode->channels[0].user_pixel_stride;
#  382|       inc1  = decode->channels[1].user_pixel_stride;
#  383|       inc2  = decode->channels[2].user_pixel_stride;

Error: CLANG_WARNING: [#def15]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:382:5: warning[deadcode.DeadStores]: Value stored to 'inc1' is never read
#  380|       h     = decode->chunk.height;
#  381|       inc0  = decode->channels[0].user_pixel_stride;
#  382|->     inc1  = decode->channels[1].user_pixel_stride;
#  383|       inc2  = decode->channels[2].user_pixel_stride;
#  384|       linc0 = decode->channels[0].user_line_stride;

Error: CLANG_WARNING: [#def16]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:383:5: warning[deadcode.DeadStores]: Value stored to 'inc2' is never read
#  381|       inc0  = decode->channels[0].user_pixel_stride;
#  382|       inc1  = decode->channels[1].user_pixel_stride;
#  383|->     inc2  = decode->channels[2].user_pixel_stride;
#  384|       linc0 = decode->channels[0].user_line_stride;
#  385|       linc1 = decode->channels[1].user_line_stride;

Error: COMPILER_WARNING (CWE-563): [#def17]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:430:21: warning[-Wunused-but-set-variable]: variable ‘inc0’ set but not used
#  430 |     int             inc0, inc1, inc2;
#      |                     ^~~~
#  428|       uint8_t *       out0, *out1, *out2;
#  429|       int             w, h;
#  430|->     int             inc0, inc1, inc2;
#  431|       int             linc0, linc1, linc2;
#  432|   

Error: COMPILER_WARNING (CWE-563): [#def18]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:430:27: warning[-Wunused-but-set-variable]: variable ‘inc1’ set but not used
#  430 |     int             inc0, inc1, inc2;
#      |                           ^~~~
#  428|       uint8_t *       out0, *out1, *out2;
#  429|       int             w, h;
#  430|->     int             inc0, inc1, inc2;
#  431|       int             linc0, linc1, linc2;
#  432|   

Error: COMPILER_WARNING (CWE-563): [#def19]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c: scope_hint: In function ‘unpack_half_to_float_3chan_planar’
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:430:33: warning[-Wunused-but-set-variable]: variable ‘inc2’ set but not used
#  430 |     int             inc0, inc1, inc2;
#      |                                 ^~~~
#  428|       uint8_t *       out0, *out1, *out2;
#  429|       int             w, h;
#  430|->     int             inc0, inc1, inc2;
#  431|       int             linc0, linc1, linc2;
#  432|   

Error: CLANG_WARNING: [#def20]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:435:5: warning[deadcode.DeadStores]: Value stored to 'inc0' is never read
#  433|       w     = decode->channels[0].width;
#  434|       h     = decode->chunk.height;
#  435|->     inc0  = decode->channels[0].user_pixel_stride;
#  436|       inc1  = decode->channels[1].user_pixel_stride;
#  437|       inc2  = decode->channels[2].user_pixel_stride;

Error: CLANG_WARNING: [#def21]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:436:5: warning[deadcode.DeadStores]: Value stored to 'inc1' is never read
#  434|       h     = decode->chunk.height;
#  435|       inc0  = decode->channels[0].user_pixel_stride;
#  436|->     inc1  = decode->channels[1].user_pixel_stride;
#  437|       inc2  = decode->channels[2].user_pixel_stride;
#  438|       linc0 = decode->channels[0].user_line_stride;

Error: CLANG_WARNING: [#def22]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRCore/unpack.c:437:5: warning[deadcode.DeadStores]: Value stored to 'inc2' is never read
#  435|       inc0  = decode->channels[0].user_pixel_stride;
#  436|       inc1  = decode->channels[1].user_pixel_stride;
#  437|->     inc2  = decode->channels[2].user_pixel_stride;
#  438|       linc0 = decode->channels[0].user_line_stride;
#  439|       linc1 = decode->channels[1].user_line_stride;

Error: COMPILER_WARNING (CWE-563): [#def23]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRUtil/ImfCheckFile.cpp:540:18: warning[-Wunused-variable]: unused variable ‘height’
#  540 |         uint64_t height = static_cast<uint64_t>(dataWindow.size().y)+1;
#      |                  ^~~~~~
#  538|           // use uint64_t for dimensions, since dataWindow+1 could overflow int storage
#  539|           //
#  540|->         uint64_t height = static_cast<uint64_t>(dataWindow.size().y)+1;
#  541|           uint64_t width = static_cast<uint64_t>(dataWindow.size().x)+1;
#  542|           int bytesPerSample = calculateBytesPerPixel(in.header());

Error: COMPILER_WARNING (CWE-563): [#def24]
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRUtil/ImfCheckFile.cpp:541:18: warning[-Wunused-variable]: unused variable ‘width’
#  541 |         uint64_t width = static_cast<uint64_t>(dataWindow.size().x)+1;
#      |                  ^~~~~
openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRUtil/ImfCheckFile.cpp: scope_hint: In instantiation of ‘bool Imf_3_1::{anonymous}::readDeepTile(T&, bool, bool) [with T = Imf_3_1::DeepTiledInputFile]’
#/builddir/build/BUILD/openexr-3.1.10-build/openexr-3.1.10/src/lib/OpenEXRUtil/ImfCheckFile.cpp:1160:34:   required from ‘bool Imf_3_1::{anonymous}::runChecks(T&, bool, bool) [with T = const char*]’
# 1160 |           gotThrow = readDeepTile( rgba, reduceMemory , reduceTime );
#      |                      ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# 1719 |     if (!threw) threw = runChecks (fileName, reduceMemory, reduceTime);
#      |                         ~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
#  539|           //
#  540|           uint64_t height = static_cast<uint64_t>(dataWindow.size().y)+1;
#  541|->         uint64_t width = static_cast<uint64_t>(dataWindow.size().x)+1;
#  542|           int bytesPerSample = calculateBytesPerPixel(in.header());
#  543|

Scan Properties

analyzer-version-clang18.1.7
analyzer-version-cppcheck2.14.2
analyzer-version-gcc14.1.1
analyzer-version-gcc-analyzer14.1.1
analyzer-version-shellcheck0.10.0
diffbase-analyzer-version-clang18.1.7
diffbase-analyzer-version-cppcheck2.14.2
diffbase-analyzer-version-gcc14.1.1
diffbase-analyzer-version-gcc-analyzer14.1.1
diffbase-analyzer-version-shellcheck0.10.0
diffbase-enabled-pluginsclang, cppcheck, gcc, shellcheck
diffbase-exit-code0
diffbase-hostip-172-16-1-155.us-west-2.compute.internal
diffbase-mock-configfedora-41-x86_64
diffbase-project-nameopenexr-3.2.4-2.fc41
diffbase-store-results-to/tmp/tmp4xv_i8va/openexr-3.2.4-2.fc41.tar.xz
diffbase-time-created2024-07-03 16:39:38
diffbase-time-finished2024-07-03 16:46:13
diffbase-toolcsmock
diffbase-tool-args'/usr/bin/csmock' '-r' 'fedora-41-x86_64' '-t' 'cppcheck,gcc,clang,shellcheck' '-o' '/tmp/tmp4xv_i8va/openexr-3.2.4-2.fc41.tar.xz' '--gcc-analyze' '/tmp/tmp4xv_i8va/openexr-3.2.4-2.fc41.src.rpm'
diffbase-tool-versioncsmock-3.5.3-1.el9
enabled-pluginsclang, cppcheck, gcc, shellcheck
exit-code0
hostip-172-16-1-155.us-west-2.compute.internal
mock-configfedora-41-x86_64
project-nameopenexr-3.1.10-5.fc40
store-results-to/tmp/tmpe5yhzj2m/openexr-3.1.10-5.fc40.tar.xz
time-created2024-07-03 16:32:46
time-finished2024-07-03 16:39:24
titleFixed defects
toolcsmock
tool-args'/usr/bin/csmock' '-r' 'fedora-41-x86_64' '-t' 'cppcheck,gcc,clang,shellcheck' '-o' '/tmp/tmpe5yhzj2m/openexr-3.1.10-5.fc40.tar.xz' '--gcc-analyze' '/tmp/tmpe5yhzj2m/openexr-3.1.10-5.fc40.src.rpm'
tool-versioncsmock-3.5.3-1.el9