View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000236 | JVT JM H.264/AVC reference software | encoder and decoder | public | 2010-10-19 10:57 | 2011-04-05 18:19 |
Reporter | Sarnath | Assigned To | Karsten Suehring | ||
Priority | normal | Severity | trivial | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Fixed in Version | JM 18.0 | ||||
Summary | 0000236: CPImage is not using the dimensions of the Chroma components properly | ||||
Description | The 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)); } } | ||||
Tags | No tags attached. | ||||
|
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. |
|
fixed |
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 |