Skip to content

Commit 2e05467

Browse files
y-guyony-guyon
committed
Add t35_identifier_length and move descr to hrsd
1 parent 9f176cb commit 2e05467

File tree

12 files changed

+304
-125
lines changed

12 files changed

+304
-125
lines changed

‎IsoLib/isoiff_tool/w32/isoiff/isoiff.dsp‎

Lines changed: 4 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎IsoLib/libisomediafile/CMakeLists.txt‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ add_library(
7171
src/H263SpecificInfoAtom.c
7272
src/HandlerAtom.c
7373
src/HEVCConfigAtom.c
74+
src/HumanReadableStreamDescriptionAtom.c
7475
src/LHEVCConfigAtom.c
7576
src/VVCConfigAtom.c
7677
src/VVCNALUConfigAtom.c

‎IsoLib/libisomediafile/linux/libisomediafile/Makefile‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,7 @@ sources = \
158158
TextMetaSampleEntry.c \
159159
XMLMetaSampleEntry.c \
160160
BitRateAtom.c \
161+
HumanReadableStreamDescriptionAtom.c \
161162
AMRWPSpecificInfoAtom.c \
162163
AMRSpecificInfoAtom.c \
163164
H263SpecificInfoAtom.c \

‎IsoLib/libisomediafile/macosx/libisomediafile/Makefile‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,7 @@ sources = \
158158
TextMetaSampleEntry.c \
159159
XMLMetaSampleEntry.c \
160160
BitRateAtom.c \
161+
HumanReadableStreamDescriptionAtom.c \
161162
AMRWPSpecificInfoAtom.c \
162163
AMRSpecificInfoAtom.c \
163164
H263SpecificInfoAtom.c \

‎IsoLib/libisomediafile/src/HumanReadableStreamDescriptionAtom.c‎

Lines changed: 164 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,164 @@
1+
/*
2+
Copyright 2024 Google LLC
3+
4+
Licensed under the Apache License, Version 2.0 (the "License");
5+
you may not use this file except in compliance with the License.
6+
You may obtain a copy of the License at
7+
8+
http://www.apache.org/licenses/LICENSE-2.0
9+
10+
Unless required by applicable law or agreed to in writing, software
11+
distributed under the License is distributed on an "AS IS" BASIS,
12+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
See the License for the specific language governing permissions and
14+
limitations under the License.
15+
*/
16+
/*
17+
$Id: HumanReadableStreamDescriptionAtom.c,v 1.0 2026/03/05 17:00:48 erapefh Exp $
18+
*/
19+
20+
#include "MP4Atoms.h"
21+
#include <stdlib.h>
22+
23+
static void destroy(MP4AtomPtr s)
24+
{
25+
MP4HumanReadableStreamDescriptionAtomPtr self;
26+
self = (MP4HumanReadableStreamDescriptionAtomPtr)s;
27+
if(self == NULL) return;
28+
29+
if(self->description)
30+
{
31+
free(self->description);
32+
self->description = NULL;
33+
}
34+
35+
if(self->super) self->super->destroy(s);
36+
}
37+
38+
static MP4Err serialize(struct MP4Atom *s, char *buffer)
39+
{
40+
MP4Err err;
41+
MP4HumanReadableStreamDescriptionAtomPtr self = (MP4HumanReadableStreamDescriptionAtomPtr)s;
42+
err = MP4NoErr;
43+
44+
err = MP4SerializeCommonBaseAtomFields(s, buffer);
45+
if(err) goto bail;
46+
buffer += self->bytesWritten;
47+
assert(self->type == MP4HumanReadableStreamDescriptionAtomType);
48+
49+
/* Write description as null-terminated UTF-8 string */
50+
if(self->description != NULL)
51+
{
52+
u32 descLen = (u32)strlen(self->description) + 1; /* Include null terminator */
53+
PUTBYTES(self->description, descLen);
54+
}
55+
else
56+
{
57+
/* Empty description: just write '\0' */
58+
u8 nullByte = 0;
59+
PUT8_V(nullByte);
60+
}
61+
62+
assert(self->bytesWritten == self->size);
63+
bail:
64+
TEST_RETURN(err);
65+
66+
return err;
67+
}
68+
69+
static MP4Err calculateSize(struct MP4Atom *s)
70+
{
71+
MP4Err err;
72+
MP4HumanReadableStreamDescriptionAtomPtr self = (MP4HumanReadableStreamDescriptionAtomPtr)s;
73+
err = MP4NoErr;
74+
75+
err = MP4CalculateBaseAtomFieldSize(s);
76+
if(err) goto bail;
77+
/* Add description size (null-terminated string) */
78+
if(self->description != NULL)
79+
{
80+
self->size += (u32)strlen(self->description) + 1; /* Including '\0' */
81+
}
82+
else
83+
{
84+
self->size += 1; /* Just '\0' */
85+
}
86+
bail:
87+
TEST_RETURN(err);
88+
89+
return err;
90+
}
91+
92+
static MP4Err createFromInputStream(MP4AtomPtr s, MP4AtomPtr proto, MP4InputStreamPtr inputStream)
93+
{
94+
MP4Err err;
95+
MP4HumanReadableStreamDescriptionAtomPtr self = (MP4HumanReadableStreamDescriptionAtomPtr)s;
96+
s64 bytesToRead;
97+
u32 descLen;
98+
u32 i;
99+
100+
if(self == NULL) BAILWITHERROR(MP4BadParamErr)
101+
err = self->super->createFromInputStream(s, proto, (char *)inputStream);
102+
if(err) goto bail;
103+
104+
/* Calculate remaining bytes to read */
105+
bytesToRead = (s64)(self->size - self->bytesRead);
106+
if(bytesToRead < 0) BAILWITHERROR(MP4BadDataErr);
107+
108+
if(bytesToRead > 0)
109+
{
110+
/* Read description (null-terminated UTF-8 string) */
111+
/* First, scan for null terminator to find description length */
112+
descLen = 0;
113+
for(i = 0; i < (u32)bytesToRead; i++)
114+
{
115+
u8 byte;
116+
err = inputStream->read8(inputStream, (u32 *)&byte, NULL);
117+
if(err) goto bail;
118+
descLen++;
119+
self->bytesRead++;
120+
if(byte == 0) break;
121+
}
122+
123+
/* Allocate and read description - we need to re-read the bytes */
124+
/* Rewind by backing up bytesRead */
125+
self->bytesRead -= descLen;
126+
err = inputStream->rewindData(inputStream, descLen, NULL);
127+
128+
self->description = (char *)calloc(descLen, 1);
129+
TESTMALLOC(self->description);
130+
err = inputStream->readData(inputStream, descLen, (char *)self->description, NULL);
131+
if(err) goto bail;
132+
self->bytesRead += descLen;
133+
}
134+
135+
bail:
136+
TEST_RETURN(err);
137+
138+
return err;
139+
}
140+
141+
MP4Err
142+
MP4CreateHumanReadableStreamDescriptionAtom(MP4HumanReadableStreamDescriptionAtomPtr *outAtom)
143+
{
144+
MP4Err err;
145+
MP4HumanReadableStreamDescriptionAtomPtr self;
146+
147+
self = (MP4HumanReadableStreamDescriptionAtomPtr)calloc(
148+
1, sizeof(MP4HumanReadableStreamDescriptionAtom));
149+
TESTMALLOC(self);
150+
151+
err = MP4CreateBaseAtom((MP4AtomPtr)self);
152+
if(err) goto bail;
153+
self->type = MP4HumanReadableStreamDescriptionAtomType;
154+
self->name = "HumanReadableStreamDescription";
155+
self->createFromInputStream = (cisfunc)createFromInputStream;
156+
self->destroy = destroy;
157+
self->calculateSize = calculateSize;
158+
self->serialize = serialize;
159+
*outAtom = self;
160+
bail:
161+
TEST_RETURN(err);
162+
163+
return err;
164+
}

‎IsoLib/libisomediafile/src/ISOSampleDescriptions.c‎

Lines changed: 39 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -46,43 +46,28 @@ MP4Err MP4ParseAtomUsingProtoList(MP4InputStreamPtr inputStream, u32 *protoList,
4646
MP4AtomPtr *outAtom);
4747

4848
#ifdef ISMACrypt
49-
u32 MP4SampleEntryProtos[] = {MP4MPEGSampleEntryAtomType,
50-
MP4VisualSampleEntryAtomType,
51-
MP4AudioSampleEntryAtomType,
52-
MP4EncAudioSampleEntryAtomType,
53-
MP4EncVisualSampleEntryAtomType,
54-
MP4XMLMetaSampleEntryAtomType,
55-
MP4TextMetaSampleEntryAtomType,
56-
MP4AMRSampleEntryAtomType,
57-
MP4AWBSampleEntryAtomType,
58-
MP4AMRWPSampleEntryAtomType,
59-
MP4H263SampleEntryAtomType,
60-
MP4RestrictedVideoSampleEntryAtomType,
61-
MP4BoxedMetadataSampleEntryType,
62-
ISOAVCSampleEntryAtomType,
63-
ISOHEVCSampleEntryAtomType,
64-
ISOVVCSampleEntryAtomTypeInBand,
65-
ISOVVCSampleEntryAtomTypeOutOfBand,
66-
ISOVVCSubpicSampleEntryAtomType,
67-
0};
49+
u32 MP4SampleEntryProtos[] = {
50+
MP4MPEGSampleEntryAtomType, MP4VisualSampleEntryAtomType,
51+
MP4AudioSampleEntryAtomType, MP4EncAudioSampleEntryAtomType,
52+
MP4EncVisualSampleEntryAtomType, MP4XMLMetaSampleEntryAtomType,
53+
MP4TextMetaSampleEntryAtomType, MP4AMRSampleEntryAtomType,
54+
MP4AWBSampleEntryAtomType, MP4AMRWPSampleEntryAtomType,
55+
MP4H263SampleEntryAtomType, MP4RestrictedVideoSampleEntryAtomType,
56+
MP4BoxedMetadataSampleEntryType, MP4T35SampleGroupEntry,
57+
ISOAVCSampleEntryAtomType, ISOHEVCSampleEntryAtomType,
58+
ISOVVCSampleEntryAtomTypeInBand, ISOVVCSampleEntryAtomTypeOutOfBand,
59+
ISOVVCSubpicSampleEntryAtomType, 0};
6860
#else
69-
u32 MP4SampleEntryProtos[] = {MP4MPEGSampleEntryAtomType,
70-
MP4VisualSampleEntryAtomType,
71-
MP4AudioSampleEntryAtomType,
72-
MP4XMLMetaSampleEntryAtomType,
73-
MP4TextMetaSampleEntryAtomType,
74-
MP4AMRSampleEntryAtomType,
75-
MP4AWBSampleEntryAtomType,
76-
MP4AMRWPSampleEntryAtomType,
77-
MP4H263SampleEntryAtomType,
78-
MP4RestrictedVideoSampleEntryAtomType,
79-
MP4BoxedMetadataSampleEntryType,
80-
ISOAVCSampleEntryAtomType,
81-
ISOHEVCSampleEntryAtomType,
82-
ISOVVCSampleEntryAtomTypeInBand,
83-
ISOVVCSampleEntryAtomTypeOutOfBand,
84-
ISOVVCSubpicSampleEntryAtomType,
85-
0};
61+
u32 MP4SampleEntryProtos[] = {
62+
MP4MPEGSampleEntryAtomType, MP4VisualSampleEntryAtomType,
63+
MP4AudioSampleEntryAtomType, MP4XMLMetaSampleEntryAtomType,
64+
MP4TextMetaSampleEntryAtomType, MP4AMRSampleEntryAtomType,
65+
MP4AWBSampleEntryAtomType, MP4AMRWPSampleEntryAtomType,
66+
MP4H263SampleEntryAtomType, MP4RestrictedVideoSampleEntryAtomType,
67+
MP4BoxedMetadataSampleEntryType, MP4T35SampleGroupEntry,
68+
ISOAVCSampleEntryAtomType, ISOHEVCSampleEntryAtomType,
69+
ISOVVCSampleEntryAtomTypeInBand, ISOVVCSampleEntryAtomTypeOutOfBand,
70+
ISOVVCSubpicSampleEntryAtomType, 0};
8671
#endif
8772

8873
MP4Err sampleEntryHToAtomPtr(MP4Handle sampleEntryH, MP4AtomPtr *entryPtr, u32 defaultType)
@@ -2505,6 +2490,7 @@ ISONewT35SampleDescription(MP4T35MetadataSampleEntryPtr *outSE, u32 dataReferenc
25052490
{
25062491
MP4Err err;
25072492
MP4T35MetadataSampleEntryPtr it35;
2493+
MP4HumanReadableStreamDescriptionAtomPtr hrsd;
25082494
u8 *identifier = NULL;
25092495
u32 identifierSize = 0;
25102496
char *description = NULL;
@@ -2520,10 +2506,23 @@ ISONewT35SampleDescription(MP4T35MetadataSampleEntryPtr *outSE, u32 dataReferenc
25202506
if(err) goto bail;
25212507
it35->dataReferenceIndex = dataReferenceIndex;
25222508

2523-
/* Set description and t35_identifier fields */
2524-
it35->description = description;
2525-
it35->t35_identifier = identifier;
2526-
it35->t35_identifier_size = identifierSize;
2509+
/* Set t35_identifier_length and t35_identifier fields */
2510+
it35->t35_identifier_length = identifierSize;
2511+
it35->t35_identifier = identifier;
2512+
2513+
if(description != NULL && description[0] != '\0')
2514+
{
2515+
err = MP4MakeLinkedList(&it35->ExtensionAtomList);
2516+
if(err) goto bail;
2517+
2518+
err = MP4CreateHumanReadableStreamDescriptionAtom(&hrsd);
2519+
if(err) goto bail;
2520+
hrsd->description = strdup(description);
2521+
TESTMALLOC(hrsd->description);
2522+
2523+
err = MP4AddListEntry((void *)hrsd, it35->ExtensionAtomList);
2524+
if(err) goto bail;
2525+
}
25272526

25282527
*outSE = it35;
25292528

‎IsoLib/libisomediafile/src/MP4Atoms.c‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -628,6 +628,11 @@ MP4Err MP4CreateAtom(u32 atomType, MP4AtomPtr *outAtom)
628628
err = MP4CreateBitRateAtom((MP4BitRateAtomPtr *)&newAtom);
629629
break;
630630

631+
case MP4HumanReadableStreamDescriptionAtomType:
632+
err = MP4CreateHumanReadableStreamDescriptionAtom(
633+
(MP4HumanReadableStreamDescriptionAtomPtr *)&newAtom);
634+
break;
635+
631636
case MP4ItemPropertiesAtomType:
632637
err = MP4CreateItemPropertiesAtom((MP4ItemPropertiesAtomPtr *)&newAtom);
633638
break;

‎IsoLib/libisomediafile/src/MP4Atoms.h‎

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,7 @@ enum
150150
MP4H263SampleEntryAtomType = MP4_FOUR_CHAR_CODE('s', '2', '6', '3'),
151151
MP4H263SpecificInfoAtomType = MP4_FOUR_CHAR_CODE('d', '2', '6', '3'),
152152
MP4BitRateAtomType = MP4_FOUR_CHAR_CODE('b', 't', 'r', 't'),
153+
MP4HumanReadableStreamDescriptionAtomType = MP4_FOUR_CHAR_CODE('h', 'r', 's', 'd'),
153154
TGPPBitRateAtomType = MP4_FOUR_CHAR_CODE('b', 'i', 't', 'r'),
154155
MP4OriginalFormatAtomType = MP4_FOUR_CHAR_CODE('f', 'r', 'm', 'a'),
155156
MP4SchemeTypeAtomType = MP4_FOUR_CHAR_CODE('s', 'c', 'h', 'm'),
@@ -868,9 +869,8 @@ typedef struct MP4T35MetadataSampleEntry
868869
{
869870
MP4_BASE_ATOM
870871
COMMON_SAMPLE_ENTRY_FIELDS
871-
char *description; /* UTF-8 string, '\0' if empty */
872-
u8 *t35_identifier; /* Variable length byte array */
873-
u32 t35_identifier_size; /* Size of t35_identifier in bytes */
872+
u32 t35_identifier_length; /* Size of t35_identifier in bytes */
873+
u8 *t35_identifier; /* Variable-length byte array */
874874
} MP4T35MetadataSampleEntry, *MP4T35MetadataSampleEntryPtr;
875875

876876
typedef struct MP4VisualSampleEntryAtom
@@ -1104,6 +1104,13 @@ typedef struct MP4BitRateAtom
11041104

11051105
} MP4BitRateAtom, *MP4BitRateAtomPtr;
11061106

1107+
typedef struct MP4HumanReadableStreamDescriptionAtom
1108+
{
1109+
MP4_BASE_ATOM
1110+
char *description; /* UTF-8 string */
1111+
1112+
} MP4HumanReadableStreamDescriptionAtom, *MP4HumanReadableStreamDescriptionAtomPtr;
1113+
11071114
typedef struct MP4SampleDescriptionAtom
11081115
{
11091116
MP4_FULL_ATOM
@@ -2454,6 +2461,8 @@ MP4Err MP4CreateAMRSpecificInfoAtom(MP4AMRSpecificInfoAtomPtr *outAtom);
24542461
MP4Err MP4CreateAMRWPSpecificInfoAtom(MP4AMRWPSpecificInfoAtomPtr *outAtom);
24552462
MP4Err MP4CreateH263SpecificInfoAtom(MP4H263SpecificInfoAtomPtr *outAtom);
24562463
MP4Err MP4CreateBitRateAtom(MP4BitRateAtomPtr *outAtom);
2464+
MP4Err
2465+
MP4CreateHumanReadableStreamDescriptionAtom(MP4HumanReadableStreamDescriptionAtomPtr *outAtom);
24572466

24582467
MP4Err MP4CreateVisualMediaHeaderAtom(MP4VolumetricVisualMediaHeaderAtomPtr *outAtom);
24592468

0 commit comments

Comments
 (0)