return NULL;
}
-static inline void
-set_nalu_datas (GstH264NalUnit * nalu)
+static gboolean
+gst_h264_parse_nalu_header (GstH264NalUnit * nalu)
{
guint8 *data = nalu->data + nalu->offset;
+ if (nalu->size < 1)
+ return FALSE;
+
nalu->type = (data[0] & 0x1f);
nalu->ref_idc = (data[0] & 0x60) >> 5;
nalu->idr_pic_flag = (nalu->type == 5 ? 1 : 0);
GST_DEBUG ("Nal type %u, ref_idc %u", nalu->type, nalu->ref_idc);
+ return TRUE;
}
/****** Parsing functions *****/
return GST_H264_PARSER_ERROR;
}
- nalu->valid = TRUE;
nalu->sc_offset = offset + off1;
nalu->offset = offset + off1 + 3;
nalu->data = (guint8 *) data;
+ nalu->size = size - nalu->offset;
+
+ if (!gst_h264_parse_nalu_header (nalu)) {
+ GST_WARNING ("error parsing \"NAL unit header\"");
+ nalu->size = 0;
+ return GST_H264_PARSER_BROKEN_DATA;
+ }
- set_nalu_datas (nalu);
+ nalu->valid = TRUE;
/* sc might have 2 or 3 0-bytes */
if (nalu->sc_offset > 0 && data[nalu->sc_offset - 1] == 00
return GST_H264_PARSER_OK;
}
- nalu->size = size - nalu->offset;
-
return GST_H264_PARSER_OK;
}
nalu->data = (guint8 *) data;
- set_nalu_datas (nalu);
-
- if (nalu->size < 2)
+ if (!gst_h264_parse_nalu_header (nalu)) {
+ GST_WARNING ("error parsing \"NAL unit header\"");
+ nalu->size = 0;
return GST_H264_PARSER_BROKEN_DATA;
+ }
nalu->valid = TRUE;