highperfocused/ffmpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

avcodec/hap : move parse_section_header to hap.c in order to be use by new bsf filter

Browse Source
This commit is contained in:
Martin Vignali 2018-03-11 19:15:56 +01:00
parent 688060fbb7
commit f869e54d22
3 changed files with 32 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
libavcodec/hap.c
View File

@ -53,3 +53,25 @@ av_cold void ff_hap_free_context(HapContext *ctx)
av_freep(&ctx->chunks); av_freep(&ctx->chunks);
av_freep(&ctx->chunk_results); av_freep(&ctx->chunk_results);
} }
int ff_hap_parse_section_header(GetByteContext *gbc, int *section_size,
enum HapSectionType *section_type)
{
if (bytestream2_get_bytes_left(gbc) < 4)
return AVERROR_INVALIDDATA;
*section_size = bytestream2_get_le24(gbc);
*section_type = bytestream2_get_byte(gbc);
if (*section_size == 0) {
if (bytestream2_get_bytes_left(gbc) < 4)
return AVERROR_INVALIDDATA;
*section_size = bytestream2_get_le32(gbc);
}
if (*section_size > bytestream2_get_bytes_left(gbc) || *section_size < 0)
return AVERROR_INVALIDDATA;
else
return 0;
}

6
libavcodec/hap.h
View File

@ -103,4 +103,10 @@ int ff_hap_set_chunk_count(HapContext *ctx, int count, int first_in_frame);
*/ */
av_cold void ff_hap_free_context(HapContext *ctx); av_cold void ff_hap_free_context(HapContext *ctx);
/* The first three bytes are the size of the section past the header, or zero
* if the length is stored in the next long word. The fourth byte in the first
* long word indicates the type of the current section. */
int ff_hap_parse_section_header(GetByteContext *gbc, int *section_size,
enum HapSectionType *section_type);
#endif /* AVCODEC_HAP_H */ #endif /* AVCODEC_HAP_H */

33
libavcodec/hapdec.c
View File

@ -43,31 +43,6 @@
#include "texturedsp.h" #include "texturedsp.h"
#include "thread.h" #include "thread.h"
/* The first three bytes are the size of the section past the header, or zero
* if the length is stored in the next long word. The fourth byte in the first
* long word indicates the type of the current section. */
static int parse_section_header(GetByteContext *gbc, int *section_size,
enum HapSectionType *section_type)
{
if (bytestream2_get_bytes_left(gbc) < 4)
return AVERROR_INVALIDDATA;
*section_size = bytestream2_get_le24(gbc);
*section_type = bytestream2_get_byte(gbc);
if (*section_size == 0) {
if (bytestream2_get_bytes_left(gbc) < 4)
return AVERROR_INVALIDDATA;
*section_size = bytestream2_get_le32(gbc);
}
if (*section_size > bytestream2_get_bytes_left(gbc) || *section_size < 0)
return AVERROR_INVALIDDATA;
else
return 0;
}
static int hap_parse_decode_instructions(HapContext *ctx, int size) static int hap_parse_decode_instructions(HapContext *ctx, int size)
{ {
GetByteContext *gbc = &ctx->gbc; GetByteContext *gbc = &ctx->gbc;
@ -78,7 +53,7 @@ static int hap_parse_decode_instructions(HapContext *ctx, int size)
while (size > 0) { while (size > 0) {
int stream_remaining = bytestream2_get_bytes_left(gbc); int stream_remaining = bytestream2_get_bytes_left(gbc);
ret = parse_section_header(gbc, &section_size, &section_type); ret = ff_hap_parse_section_header(gbc, &section_size, &section_type);
if (ret != 0) if (ret != 0)
return ret; return ret;
@ -159,7 +134,7 @@ static int hap_parse_frame_header(AVCodecContext *avctx)
const char *compressorstr; const char *compressorstr;
int i, ret; int i, ret;
ret = parse_section_header(gbc, &ctx->texture_section_size, &section_type); ret = ff_hap_parse_section_header(gbc, &ctx->texture_section_size, &section_type);
if (ret != 0) if (ret != 0)
return ret; return ret;
@ -190,7 +165,7 @@ static int hap_parse_frame_header(AVCodecContext *avctx)
} }
break; break;
case HAP_COMP_COMPLEX: case HAP_COMP_COMPLEX:
ret = parse_section_header(gbc, &section_size, &section_type); ret = ff_hap_parse_section_header(gbc, &section_size, &section_type);
if (ret == 0 && section_type != HAP_ST_DECODE_INSTRUCTIONS) if (ret == 0 && section_type != HAP_ST_DECODE_INSTRUCTIONS)
ret = AVERROR_INVALIDDATA; ret = AVERROR_INVALIDDATA;
if (ret == 0) if (ret == 0)
@ -342,7 +317,7 @@ static int hap_decode(AVCodecContext *avctx, void *data,
/* check for multi texture header */ /* check for multi texture header */
if (ctx->texture_count == 2) { if (ctx->texture_count == 2) {
ret = parse_section_header(&ctx->gbc, &section_size, &section_type); ret = ff_hap_parse_section_header(&ctx->gbc, &section_size, &section_type);
if (ret != 0) if (ret != 0)
return ret; return ret;
if ((section_type & 0x0F) != 0x0D) { if ((section_type & 0x0F) != 0x0D) {