| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
This is the case for Opus files with an ogg extension.
|
|
|
|
|
|
|
| |
1.12 (#1012)
* fix error: duplicate volatile
* fix volatile ATOMIC_INT in constructor
|
|
|
|
| |
Change the size check to use isEmpty() as requested.
|
|
|
|
|
| |
UBSan spotted an integer overflow on the line `dataPos += size`, so add
a bounds check to the size that we read.
|
|
|
|
|
|
| |
The while loop in this function assumes that `data.end() - 1` is less
than `data.end()`, which isn't the case if `data` is empty since
`data.end()` can be a nullptr.
|
|
|
|
|
| |
If `data` is an allocation of only 3 bytes (MP+) then `data[3]` is a
read past the end of the buffer.
|
| |
|
|
|
|
|
|
|
|
| |
`pos`, `valLegnth`, and `data.size()` are all unsigned types so we have
to do a little dance to correctly bounds check them without overflow.
Without this we can get stuck in an infinite loop due to 'pos'
overflowing back to the start of the data.
|
|
|
|
|
| |
If the provided vector is empty then `data[0]` dereferences a nullptr,
so add a check that this won't be the case before reading the encoding.
|
|
|
|
| |
Without this we can crash trying to dereference parts of `header`.
|
| |
|
|
|
|
| |
Closes #1005
|
|
|
|
|
|
| |
The options WITH_ASF and WITH_MP4 where remove in commit
dd846904cbc1ef3ee628d77f0c9df88ef8967816 back in 2011. It's time to
remove the leftovers :-)
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
| |
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
|
|
| |
ENABLE_STATIC error has been there since 2014, that is long enough.
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
| |
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
|
|
| |
Well-established CMake standard for installation directories.
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
|
|
|
| |
Small line decrease, but also easier to read what is happening.
Now all dependencies can be read from the root CMakeLists.txt file.
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
|
|
|
|
|
|
|
| |
BUILD_TESTING is default enabled, which is a good default anyway.
Move the CppUnit check to the root CMakeLists.txt, simpler and clearer.
BUILD_TESTS is obsolete.
The need for BUILD_SHARED_LIBS=OFF for testing is not clear, it works on Linux.
But I kept it in the instructions for now.
Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
|
|
|
|
| |
This also makes things look non-crappy on high-DPI monitors.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
The change from "fileref: Use user defined resolvers on streams"
does not work with the Windows FileName implementation, and
TestFileRef::testFileResolver fails.
|
| |
|
| |
|
| |
|
|\
| |
| | |
More unit tests and bug fixes
|
| |
| |
| |
| |
| |
| | |
The support for MusicBrainz properties is enhanced with "ARTISTS", "ASIN",
"RELEASECOUNTRY", "RELEASESTATUS", "RELEASETYPE", "MUSICBRAINZ_RELEASETRACKID",
"ORIGINALDATE" on APE, ASF, MP4, ID3v2, and Xiph tags.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
Now it is handled in the same way as for other tags with a
TagUnion. Without this patch, WavPack files without tags cannot
be edited via a FileRef.
|
| |
| |
| |
| |
| |
| |
| | |
For an ID3v2 "DJMIXER" property, the "DJ-MIX" TIPL role must be used.
For an ID3v2 "MIXER" property, the "MIX" TIPL role must be used.
Otherwise it will not work when reading the tag and creating
properties from the wrong TIPL roles.
|
| | |
|
| |
| |
| |
| | |
Frame::setData() has to be used instead of PrivateFrame::setData().
|
| | |
|
| | |
|
|/
|
|
|
| |
This not only increased the test coverage but also revealed some
bugs which are fixed in the following commits.
|
| |
|
| |
|
|
|
|
|
| |
These can be merged back into master from the branch
add_dsf_and_dsdiff_file_formats.
|
|\
| |
| |
| | |
Correctly read and write multiple ID3v2.3.0 genres (#988)
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
As described in id3v2.3.0.txt (4.2.1, TCON), multiple genres are
only possible as references to ID3v1 genres with an optional
refinement as a text. When downgrading multiple genres from
ID3v2.4.0, they are now converted to numbers when possible and
the first genre text without ID3v1 reference is added as a
refinement. The keywords RX and CR are supported too.
|
| |
| |
| |
| |
| |
| | |
As described in id3v2.3.0.txt (4.2.1, TCON), now multiple genres
are possible, e.g. "(51)(39)". Additionally, support the keywords
RX and CR.
|
|\ \
| | |
| | |
| | | |
WAV: Support subformat in WAVE_FORMAT_EXTENSIBLE (#850)
|
| | |
| | |
| | |
| | | |
uint8we.wav: http://www-mmsp.ece.mcgill.ca/Documents/AudioFormats/WAVE/Samples/AFsp/M1F1-uint8WE-AFsp.wav
|
| | | |
|