Community
cancel
Showing results for 
Search instead for 
Did you mean: 
Bill_Moon
Beginner
86 Views

DPD200267469 Incorrect decode of YCC444 JPEG with restart marker

I'd like more information on this bug fix, please. Our application does JPEG decoding with restart markers
0 Kudos
6 Replies
Chao_Y_Intel
Employee
86 Views

Hello,

We noticed one bug when handling one restart marker image. It happens on In the file src\io\uic_io\src\membuffin.cpp, at at line ~ 58

Change : if(m_currpos + offset >= m_buflen || m_currpos + offset < 0)
To: if(m_currpos + offset > m_buflen || m_currpos + offset < 0)

This was fixed in the latest release.

Thanks,
Chao

Thomas_Jensen1
Beginner
86 Views

You write this was fixed in the latest relase. Do you mean w_ipp-samples_p_7.0.7.064.zip? In this package, membuffin.cpp does not have the above fix.


Sergey_K_Intel
Employee
86 Views

Thomas,
You are right. That fix haven't been copied from development branch to release branch. Sorry.
Bill, could you update your source file manually and look if this solves your issue?
If not, please provide us with more details or sample file.
Regards,
Sergey
Thomas_Jensen1
Beginner
86 Views

I have put in the fix manually.

However, I get a feeling that many of the other IPP 7 Update 7 documented fixes are actually not in the samples download. You see, I always do a file-by-file comparison between releases of the ipp samples, and I only found very few changes.

Please check...
Bill_Moon
Beginner
86 Views

Hi,

After looking into this, I believe that someone on my team was the one who submitted the bug report (#654054) for this. We received the patch for this problem back in 12/2011.
I'm concerned that your release process allows a bug to bemarkedas fixed, without a verification that it actually was fixed. It makes me wonder what else was put in the release notes, without actually being there.
Chao_Y_Intel
Employee
86 Views

Hello Bill,

It was a mistake that the CQ was put into the bug fix list. The fix is planned to add into the future release. Sorry for the confusion here. Meanwhile, we are internally checking the process, soit could provide more accurate fix list.

As Sergey suggested, it still needs to manually change some source code for the fix on the IPP 7.0.7 at this moment.

Thanks,
Chao

Reply