View Issue Details

IDProjectCategoryView StatusLast Update
0000236JVT JM H.264/AVC reference softwareencoder and decoderpublic2011-04-05 18:19
ReporterSarnath Assigned ToKarsten Suehring  
PrioritynormalSeveritytrivialReproducibilityalways
Status resolvedResolutionfixed 
Fixed in VersionJM 18.0 
Summary0000236: CPImage is not using the dimensions of the Chroma components properly
DescriptionThe 3rd memcpy in the code below is using the frame dimensions of the "1"st component instead of the "2nd" component.

static inline void CPImage(ImageData *imgOut, ImageData *imgIn)
{
  memcpy(imgOut->frm_data[0][0], imgIn->frm_data[0][0], imgIn->format.height[0] * imgIn->format.width[0] * sizeof (imgpe
l));

  if (imgIn->format.yuv_format != YUV400)
  {
    memcpy(imgOut->frm_data[1][0], imgIn->frm_data[1][0], imgIn->format.height[1] * imgIn->format.width[1] * sizeof (img
pel));
    memcpy(imgOut->frm_data[2][0], imgIn->frm_data[2][0], imgIn->format.height[1] * imgIn->format.width[1] * sizeof (img
pel));
  }
}
TagsNo tags attached.

Activities

Karsten Suehring

2011-03-30 17:54

administrator   ~0000409

This seems to be a problem in the whole img_process.c file. It is probably not a problem since in all used chroma formats the second and third component have the same size, but it still should be fixed.

Karsten Suehring

2011-04-05 18:19

administrator   ~0000425

fixed

Issue History

Date Modified Username Field Change
2010-10-19 10:57 Sarnath New Issue
2011-03-30 17:54 Karsten Suehring Note Added: 0000409
2011-03-30 17:54 Karsten Suehring Status new => confirmed
2011-04-05 18:19 Karsten Suehring Note Added: 0000425
2011-04-05 18:19 Karsten Suehring Status confirmed => resolved
2011-04-05 18:19 Karsten Suehring Fixed in Version => JM 18.0
2011-04-05 18:19 Karsten Suehring Resolution open => fixed
2011-04-05 18:19 Karsten Suehring Assigned To => Karsten Suehring