summaryrefslogtreecommitdiff
path: root/libavformat/rtpdec_mpa_robust.c
Commit message (Collapse)AuthorAge
* avformat: Forward errors where possibleAndreas Rheinhardt2019-12-12
| | | | | | | | | | It is not uncommon to find code where the caller thinks to know better what the return value should be than the callee. E.g. something like "if (av_new_packet(pkt, size) < 0) return AVERROR(ENOMEM);". This commit changes several instances of this to instead forward the actual error. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* lavf/rtpdec*: Constify all RTPDynamicProtocolHandler.Carl Eugen Hoyos2018-02-14
| | | | Reviewed-by: Muhammad Faiz
* avformat/rtpdec_mpa_robust: change assignment to inequality test in conditionalGanesh Ajjanagadde2015-11-17
| | | | | | | | | | | In the spirit of commit 8199908fdf9b3797cceaea9d1e2fc09d02ef7309, likely typo originally. Found by enabling -Wparentheses on clang 3.7 and running a manual audit. To the best of my knowledge, no such instances remain. Reviewed-by: Michael Niedermayer <michael@niedermayer.cc> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
* Merge commit 'ce70f28a1732c74a9cd7fec2d56178750bd6e457'Hendrik Leppkes2015-10-27
|\ | | | | | | | | | | | | * commit 'ce70f28a1732c74a9cd7fec2d56178750bd6e457': avpacket: Replace av_free_packet with av_packet_unref Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * avpacket: Replace av_free_packet with av_packet_unrefLuca Barbato2015-10-26
| | | | | | | | | | | | | | `av_packet_unref` matches the AVFrame ref-counted API and can be used as a drop in replacement. Deprecate `av_free_packet`.
* | Merge commit '2a66a580678dd9401f4d95e01e0958ca51864b6f'Michael Niedermayer2015-03-04
|\| | | | | | | | | | | | | | | | | | | | | * commit '2a66a580678dd9401f4d95e01e0958ca51864b6f': rtpdec_mpa_robust: Fix incrementing split_pos Conflicts: libavformat/rtpdec_mpa_robust.c See: ebf1f512e9916040dd96fa9f789ed4be5a39c349 Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec_mpa_robust: Fix incrementing split_posMartin Storsjö2015-03-04
| | | | | | | | | | | | | | | | | | This fixes an oversight in 96084251, in a refactoring done on top of Gilles' original patch. Pointed out by Gilles Chanteperdrix. Signed-off-by: Martin Storsjö <martin@martin.st>
* | avformat/rtpdec_mpa_robust: fix commit 96084251e57d1738fde02a2b0d37ca609d9efd71Gilles Chanteperdrix2015-03-02
| | | | | | | | | | | | | | | | | | Commit 96084251e57d1738fde02a2b0d37ca609d9efd71 introduced a change in the parser implementation which broke it. Restore the original implementation. Signed-off-by: Gilles Chanteperdrix <gilles.chanteperdrix@xenomai.org> Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
* | Merge commit 'd594dbecce5f3af7f80bbf5bb1b516d740b69ccd'Michael Niedermayer2015-02-25
|\| | | | | | | | | | | | | * commit 'd594dbecce5f3af7f80bbf5bb1b516d740b69ccd': rtpdec: Rename the free method to close Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec: Rename the free method to closeMartin Storsjö2015-02-24
| | | | | | | | | | | | | | | | | | Many of these functions were named foo_free_context, and since the functions no longer should free the context itself, only allocated elements within it, the previous naming was slightly misleading. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit '199fb40278146c5bb162990c66ad3cd561abc780'Michael Niedermayer2015-02-25
|\| | | | | | | | | | | | | | | | | | | | | | | * commit '199fb40278146c5bb162990c66ad3cd561abc780': rtpdec: Use ffio_free_dyn_buf Conflicts: libavformat/rtpdec_latm.c libavformat/rtpdec_svq3.c libavformat/rtpdec_xiph.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec: Use ffio_free_dyn_bufMartin Storsjö2015-02-24
| | | | | | | | Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit 'bb4a310bb85f43e62240145a656b1e5285b14239'Michael Niedermayer2015-02-25
|\| | | | | | | | | | | | | | | | | | | | | | | | | * commit 'bb4a310bb85f43e62240145a656b1e5285b14239': rtpdec: Don't free the payload context in the .free function Conflicts: libavformat/rtpdec_latm.c libavformat/rtpdec_mpeg4.c libavformat/rtpdec_mpegts.c libavformat/rtpdec_xiph.c Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec: Don't free the payload context in the .free functionMartin Storsjö2015-02-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes it more consistent with depacketizers that don't have any .free function at all, where the payload context is freed by the surrounding framework. Always free the context in the surrounding framework, having the individual depacketizers only free any data they've specifically allocated themselves. This is similar to how this works for demuxer/muxers/codecs - a component shouldn't free the priv_data that the framework has allocated for it. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit '88434f9725e7c9484dcbcf323566ae88a2904f32'Michael Niedermayer2015-02-24
|\| | | | | | | | | | | | | * commit '88434f9725e7c9484dcbcf323566ae88a2904f32': rtpdec: Remove unnecessary inline attributes Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec: Remove unnecessary inline attributesMartin Storsjö2015-02-24
| | | | | | | | | | | | | | These functions are far from performance critical, so there's no point in marking them as inline. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit '5d8cae45737bed6239bd6b6e0698802dbe1463c8'Michael Niedermayer2015-02-24
|\| | | | | | | | | | | | | * commit '5d8cae45737bed6239bd6b6e0698802dbe1463c8': rtpdec: Get rid of all trivial .alloc/.free functions Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec: Get rid of all trivial .alloc/.free functionsMartin Storsjö2015-02-24
| | | | | | | | Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit '2b982e92f42a6e661d90b12b6592cd13cae496d4'Michael Niedermayer2015-02-24
|\| | | | | | | | | | | | | * commit '2b982e92f42a6e661d90b12b6592cd13cae496d4': rtpdec: Set need_parsing via a handler field Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec: Set need_parsing via a handler fieldMartin Storsjö2015-02-24
| | | | | | | | | | | | | | This avoids implementing a full function just to set this one field. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit 'b651c9139e1ab222d5aab9151dcd7d6e40e49885'Michael Niedermayer2015-02-24
|\| | | | | | | | | | | | | * commit 'b651c9139e1ab222d5aab9151dcd7d6e40e49885': rtpdec_mpa_robust: Move .enc_name to the start of the struct Merged-by: Michael Niedermayer <michaelni@gmx.at>
| * rtpdec_mpa_robust: Move .enc_name to the start of the structMartin Storsjö2015-02-24
| | | | | | | | | | | | This makes it match the other depacketizers. Signed-off-by: Martin Storsjö <martin@martin.st>
* | Merge commit '96084251e57d1738fde02a2b0d37ca609d9efd71'Michael Niedermayer2015-02-22
|/ | | | | | | | | | | | * commit '96084251e57d1738fde02a2b0d37ca609d9efd71': libavformat: add robust MPEG audio depacketization (RFC 5219) Conflicts: Changelog libavformat/version.h See: 22470510d1f9441e848bbe107c7963b6d492b47f Merged-by: Michael Niedermayer <michaelni@gmx.at>
* libavformat: add robust MPEG audio depacketization (RFC 5219)Gilles Chanteperdrix2015-02-21
Signed-off-by: Martin Storsjö <martin@martin.st>
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-01 09:05:42 +0000