Commit 4a294775 authored by Maksim Shabunin's avatar Maksim Shabunin

Coverity: fixed uninitialized field in ExifReader

parent a9607c85
...@@ -270,7 +270,11 @@ int JpegDecoder::getOrientation() ...@@ -270,7 +270,11 @@ int JpegDecoder::getOrientation()
ExifReader reader( m_filename ); ExifReader reader( m_filename );
if( reader.parse() ) if( reader.parse() )
{ {
orientation = reader.getTag( ORIENTATION ).field_u16;//orientation is unsigned short, so check field_u16 ExifEntry_t entry = reader.getTag( ORIENTATION );
if (entry.tag != INVALID_TAG)
{
orientation = entry.field_u16; //orientation is unsigned short, so check field_u16
}
} }
return orientation; return orientation;
......
...@@ -54,14 +54,14 @@ namespace cv ...@@ -54,14 +54,14 @@ namespace cv
ExifEntry_t::ExifEntry_t() : ExifEntry_t::ExifEntry_t() :
field_float(0), field_double(0), field_u32(0), field_s32(0), field_float(0), field_double(0), field_u32(0), field_s32(0),
tag(0), field_u16(0), field_s16(0), field_u8(0), field_s8(0) tag(INVALID_TAG), field_u16(0), field_s16(0), field_u8(0), field_s8(0)
{ {
} }
/** /**
* @brief ExifReader constructor * @brief ExifReader constructor
*/ */
ExifReader::ExifReader(std::string filename) : m_filename(filename) ExifReader::ExifReader(std::string filename) : m_filename(filename), m_format(NONE)
{ {
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment