From 290799f8a54e2b1531b6930ed988c5ddc6a2eb42 Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Wed, 27 Sep 2017 12:33:36 -0400 Subject: Remove error string from CGifContext This string is set, but never used. BUG=pdfium:907 Change-Id: I57555d0dd3670b245554ff2d7163d1d04bcdb7e8 Reviewed-on: https://pdfium-review.googlesource.com/14930 Reviewed-by: dsinclair Commit-Queue: Ryan Harrison --- core/fxcodec/lgif/cgifcontext.cpp | 42 +++++++++++++++++++-------------------- core/fxcodec/lgif/cgifcontext.h | 2 -- 2 files changed, 20 insertions(+), 24 deletions(-) diff --git a/core/fxcodec/lgif/cgifcontext.cpp b/core/fxcodec/lgif/cgifcontext.cpp index d1b9b44583..af1895316f 100644 --- a/core/fxcodec/lgif/cgifcontext.cpp +++ b/core/fxcodec/lgif/cgifcontext.cpp @@ -39,15 +39,10 @@ CGifContext::CGifContext(CCodec_GifModule* gif_module, global_sort_flag(0), global_color_resolution(0), img_pass_num(0) { - memset(m_szLastError, 0, sizeof(m_szLastError)); } CGifContext::~CGifContext() {} -void CGifContext::AddError(const char* err_msg) { - strncpy(m_szLastError, err_msg, GIF_MAX_ERROR_SIZE - 1); -} - void CGifContext::RecordCurrentPosition(uint32_t* cur_pos_ptr) { m_pDelegate->GifRecordCurrentPosition(*cur_pos_ptr); } @@ -80,15 +75,15 @@ GifDecodeStatus CGifContext::ReadHeader() { return GifDecodeStatus::Unfinished; if (strncmp(gif_header_ptr->signature, GIF_SIGNATURE, 3) != 0 || - gif_header_ptr->version[0] != '8' || gif_header_ptr->version[2] != 'a') { - AddError("Not A Gif Image"); + gif_header_ptr->version[0] != '8' || gif_header_ptr->version[2] != 'a') return GifDecodeStatus::Error; - } + GifLocalScreenDescriptor* gif_lsd_ptr = nullptr; if (!ReadData(reinterpret_cast(&gif_lsd_ptr), 7)) { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + if (gif_lsd_ptr->global_flags.global_pal) { global_pal_exp = gif_lsd_ptr->global_flags.pal_bits; uint32_t global_pal_size = unsigned(2 << global_pal_exp) * 3u; @@ -97,11 +92,13 @@ GifDecodeStatus CGifContext::ReadHeader() { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + global_sort_flag = gif_lsd_ptr->global_flags.sort_flag; global_color_resolution = gif_lsd_ptr->global_flags.color_resolution; m_GlobalPalette.resize(global_pal_size / 3); memcpy(m_GlobalPalette.data(), global_pal_ptr, global_pal_size); } + width = static_cast( GetWord_LSBFirst(reinterpret_cast(&gif_lsd_ptr->width))); height = static_cast( @@ -186,6 +183,7 @@ GifDecodeStatus CGifContext::GetFrame() { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + SaveDecodingStatus(GIF_D_STATUS_IMG_DATA); skip_size_org = skip_size; if (!ReadData(&data_size_ptr, 1)) @@ -214,10 +212,9 @@ GifDecodeStatus CGifContext::LoadFrame(int32_t frame_num) { uint32_t skip_size_org = skip_size; GifImage* gif_image_ptr = m_Images[frame_num].get(); uint32_t gif_img_row_bytes = gif_image_ptr->m_ImageInfo.width; - if (gif_img_row_bytes == 0) { - AddError("Error Invalid Number of Row Bytes"); + if (gif_img_row_bytes == 0) return GifDecodeStatus::Error; - } + if (decode_status == GIF_D_STATUS_TAIL) { gif_image_ptr->m_ImageRowBuf.resize(gif_img_row_bytes); GifGraphicControlExtension* gif_img_gce_ptr = @@ -238,7 +235,6 @@ GifDecodeStatus CGifContext::LoadFrame(int32_t frame_num) { -1, 0, gif_image_ptr->m_ImageInfo.local_flags.interlace); if (!bRes) { gif_image_ptr->m_ImageRowBuf.clear(); - AddError("Error Read Record Position Data"); return GifDecodeStatus::Error; } } else { @@ -256,13 +252,12 @@ GifDecodeStatus CGifContext::LoadFrame(int32_t frame_num) { gif_image_ptr->m_ImageInfo.local_flags.interlace); if (!bRes) { gif_image_ptr->m_ImageRowBuf.clear(); - AddError("Error Read Record Position Data"); return GifDecodeStatus::Error; } } + if (gif_image_ptr->image_code_exp > GIF_MAX_LZW_EXP) { gif_image_ptr->m_ImageRowBuf.clear(); - AddError("Error Invalid Code Size"); return GifDecodeStatus::Error; } @@ -282,6 +277,7 @@ GifDecodeStatus CGifContext::LoadFrame(int32_t frame_num) { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + if (!m_ImgDecoder.get()) m_ImgDecoder = CFX_LZWDecoder::Create(!gif_image_ptr->m_LocalPalettes.empty() @@ -374,7 +370,6 @@ GifDecodeStatus CGifContext::LoadFrame(int32_t frame_num) { } SaveDecodingStatus(GIF_D_STATUS_TAIL); } - AddError("Decode Image Data Error"); return GifDecodeStatus::Error; } @@ -423,6 +418,7 @@ GifDecodeStatus CGifContext::DecodeExtension() { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + cmt_data.clear(); while (*data_size_ptr != GIF_BLOCK_TERMINAL) { uint8_t data_size = *data_size_ptr; @@ -431,6 +427,7 @@ GifDecodeStatus CGifContext::DecodeExtension() { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + cmt_data += ByteString(data_ptr, data_size); } break; @@ -445,6 +442,7 @@ GifDecodeStatus CGifContext::DecodeExtension() { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + while (*data_size_ptr != GIF_BLOCK_TERMINAL) { if (!ReadData(&data_ptr, *data_size_ptr) || !ReadData(&data_size_ptr, 1)) { @@ -489,10 +487,9 @@ GifDecodeStatus CGifContext::DecodeExtension() { } GifDecodeStatus CGifContext::DecodeImageInfo() { - if (width == 0 || height == 0) { - AddError("No Image Header Info"); + if (width == 0 || height == 0) return GifDecodeStatus::Error; - } + uint32_t skip_size_org = skip_size; GifImageInfo* gif_img_info_ptr = nullptr; if (!ReadData(reinterpret_cast(&gif_img_info_ptr), 9)) @@ -509,10 +506,9 @@ GifDecodeStatus CGifContext::DecodeImageInfo() { GetWord_LSBFirst(reinterpret_cast(&gif_img_info_ptr->height)); gif_image->m_ImageInfo.local_flags = gif_img_info_ptr->local_flags; if (gif_image->m_ImageInfo.left + gif_image->m_ImageInfo.width > width || - gif_image->m_ImageInfo.top + gif_image->m_ImageInfo.height > height) { - AddError("Image Data Out Of LSD, The File May Be Corrupt"); + gif_image->m_ImageInfo.top + gif_image->m_ImageInfo.height > height) return GifDecodeStatus::Error; - } + GifLocalFlags* gif_img_info_lf_ptr = &gif_img_info_ptr->local_flags; if (gif_img_info_lf_ptr->local_pal) { gif_image->local_pallette_exp = gif_img_info_lf_ptr->pal_bits; @@ -522,15 +518,18 @@ GifDecodeStatus CGifContext::DecodeImageInfo() { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + gif_image->m_LocalPalettes = std::vector(loc_pal_size / 3); std::copy(loc_pal_ptr, loc_pal_ptr + loc_pal_size, reinterpret_cast(gif_image->m_LocalPalettes.data())); } + uint8_t* code_size_ptr = nullptr; if (!ReadData(&code_size_ptr, 1)) { skip_size = skip_size_org; return GifDecodeStatus::Unfinished; } + gif_image->image_code_exp = *code_size_ptr; RecordCurrentPosition(&gif_image->image_data_pos); gif_image->image_data_pos += skip_size; @@ -547,5 +546,4 @@ GifDecodeStatus CGifContext::DecodeImageInfo() { void CGifContext::DecodingFailureAtTailCleanup(GifImage* gif_image_ptr) { gif_image_ptr->m_ImageRowBuf.clear(); SaveDecodingStatus(GIF_D_STATUS_TAIL); - AddError("Decode Image Data Error"); } diff --git a/core/fxcodec/lgif/cgifcontext.h b/core/fxcodec/lgif/cgifcontext.h index c0a8a434bc..2573a2e470 100644 --- a/core/fxcodec/lgif/cgifcontext.h +++ b/core/fxcodec/lgif/cgifcontext.h @@ -22,7 +22,6 @@ class CGifContext : public CCodec_GifModule::Context { CCodec_GifModule::Delegate* pDelegate); ~CGifContext() override; - void AddError(const char* err_msg); void RecordCurrentPosition(uint32_t* cur_pos_ptr); void ReadScanline(int32_t row_num, uint8_t* row_buf); bool GetRecordPosition(uint32_t cur_pos, @@ -65,7 +64,6 @@ class CGifContext : public CCodec_GifModule::Context { uint8_t global_sort_flag; uint8_t global_color_resolution; uint8_t img_pass_num; - char m_szLastError[GIF_MAX_ERROR_SIZE]; private: uint8_t* ReadData(uint8_t** des_buf_pp, uint32_t data_size); -- cgit v1.2.3