1. 04 Nov, 2016 3 commits
    • Ralph Giles's avatar
      mingw: update to opus 1.1.3. · bfb3c96d
      Ralph Giles authored
      Build against latest stable upstream release.
      bfb3c96d
    • Ralph Giles's avatar
      mingw: Correct README typo. · 8da7aac7
      Ralph Giles authored
      8da7aac7
    • Ralph Giles's avatar
      Make dot optional in generating documentation. · bff125da
      Ralph Giles authored
      Different distributions of doxygen have different default
      values of HAVE_DOT setting, so we need to pick a specific
      setting to avoid 'missing dot' warnings on some platforms.
      
      Doxygen uses it to generate inclusion graphs for our various
      header files, which is somewhat useful, but not essential.
      We therefore enable dot if it's present (usually through
      the parent graphviz package) but disable it if it's not
      available, silencing the warning, but not giving uniform
      results.
      bff125da
  2. 28 Sep, 2016 1 commit
  3. 21 Sep, 2016 4 commits
  4. 16 Sep, 2016 1 commit
    • Timothy B. Terriberry's avatar
      Fix MSVC warnings. · d0c82543
      Timothy B. Terriberry authored
      Some of these pointed to real potential overflows (given arbitrary
       inputs by the calling application).
      I was sad about stripping const qualifiers from the struct addrinfo
       pointers, but MSVC seems to erroneously think that an array of
       pointers to constant data is itself a pointer to constant data (or
       maybe that it is not compatible with a const void *?), and
       converting the memmove()s to for loops triggered an erroneous
       warning about out-of-bounds array accesses in gcc (but on only one
       of the two identical loops).
      d0c82543
  5. 29 Jul, 2016 4 commits
  6. 27 Jul, 2016 4 commits
  7. 08 Jul, 2016 2 commits
  8. 06 Jul, 2016 4 commits
  9. 04 Jul, 2016 7 commits
    • Timothy B. Terriberry's avatar
      d21816d6
    • Timothy B. Terriberry's avatar
      Fix free with uninitialized data in opus_tags_parse(). · 72f4f8a6
      Timothy B. Terriberry authored
      If the parsing fails before all comments are filled in, we will
       attempt to free any binary metadata at the position one past the
       last comment, which will be uninitialized.
      Introduced in commit 0221ca95.
      72f4f8a6
    • Timothy B. Terriberry's avatar
      Add missing NULL check to opus_tags_parse(). · bd607f5c
      Timothy B. Terriberry authored
      According to the API, you can pass in a NULL OpusTags object to
       simply check if the comment packet is valid, without storing the
       parsed results.
      However, the additions to store binary metadata in commit
       0221ca95 did not check for this.
      
      Fixes Coverity CID 149873.
      bd607f5c
    • Timothy B. Terriberry's avatar
      Fix NULL check in opus_tags_add_comment(). · 66a8c158
      Timothy B. Terriberry authored
      In 0221ca95 the allocation result went from being stored
       directly in "_tags->user_comments[ncomments]" to being stored in
       the temporary "comment".
      However, the NULL check for allocation failure was not updated to
       match.
      This meant this function would almost always fail, unless you had
       added binary metadata first.
      
      Fixes Coverity CID 149874.
      66a8c158
    • Timothy B. Terriberry's avatar
      Fix skipping logic for multiplexed non-Opus pages. · 78cd9bcf
      Timothy B. Terriberry authored
      This bug appears to have been present since the original code
       import.
      This was a "clever" rearrangement of the control flow from the
       _fetch_and_process_packet() in vorbisfile to use a do ... while(0)
       instead of a "while(1)".
      However, this also makes "continue" equivalent to "break": it does
       not actually go back to the top of the loop, because the loop
       condition is false.
      
      This bug was harmless, because ogg_stream_pagein() then refuses to
       ingest a page with the wrong serialno, but we can simplify things
       by fixing it.
      The "not strictly necessary" loop is now completely unnecessary.
      The extra checks that existed in vorbisfile have all been moved to
       later in the main loop, so we can just continue that one directly,
       with no wasted work, instead of embedding a smaller loop inside.
      
      Fixes Coverity CID 149875.
      78cd9bcf
    • Timothy B. Terriberry's avatar
      Note small inaccuracy in bitrate tracking. · 73909d7d
      Timothy B. Terriberry authored
      In the non-seekable case, we'll undercount some bytes at the start
       of a new link.
      Still thinking about the best way to address this, but leaving a
       comment so I don't forget.
      73909d7d
    • Timothy B. Terriberry's avatar
      Should a BOS page with no packets be an error? · 3abc3541
      Timothy B. Terriberry authored
      Going with "no" for now, but leave a reminder in the source code
       that this is a debatable question.
      3abc3541
  10. 26 Jun, 2016 1 commit
  11. 19 Jun, 2016 1 commit
  12. 16 Jun, 2016 2 commits
  13. 01 Jun, 2016 2 commits
  14. 06 Jan, 2016 1 commit
  15. 05 Jan, 2016 2 commits
  16. 04 Jan, 2016 1 commit