Signed-off-by: NMans Rullgard <mans@mansr.com>

This adds two protocols, but one of them is an internal implementation
detail just used as an abstraction layer/generalization in the code. The
RTMPT protocol implementation uses rtmphttp:// as an alternative to the
tcp:// protocol. This allows moving most of the lower level logic out
from the higher level generic rtmp code.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This can happen if doing a new request using the same socket,
but the new request failed, which clears the urlcontext.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This was missed in the previous cleanup patch.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Also return a proper error code in these cases.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Add spaces around operators, fix brace placement and whitespace to
match K&R style, vertically align code, remove redundant != 0 and
convert x == 0 into !x, drop useless braces.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This can easily happen when the caller is using a custom AVIOContext.

Behave as if the filename was an empty string in this case.

CC: libav-stable@libav.org

Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

We generally do not declare variables within for statements and
there are compilers that choke on such constructs.

This makes sure all incoming packets are read and handled (and reacted
to) while sending an FLV stream over RTMP to a server. If there were
enough incoming data to fill the TCP buffers, this could potentially
make things block at unexpected places. For the upcoming RTMPT support,
we need to consume all incoming data before we can send the next
request.
Signed-off-by: NMartin Storsjö <martin@martin.st>

While there is no reason for starting a frame with anything else
than a Mode A packet, some senders seem to consistently use Mode B
packets for everything. This fixes depacketization of such streams.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

This factorizes existing code into a new function gen_buffer_time(),
which generates the client buffer time message and sends it to the
server.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Fixes a possible invalid write, found by Nicolas George.

This is more consistent with naming in the rest of Libav.

It will allow adding new fields to it without ABI breaks.

This will allow making AVFilterPad opaque for the calling apps, since
those are the only two fields that can be useful to the users.

They are only useful inside filters and we don't allow user filters for
now.

Fixes Bugzilla #279.
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

This fixes builds on platforms without strtok_r (windows).
Signed-off-by: NMartin Storsjö <martin@martin.st>

Fixes infinite loop in FLAC decoding in case of a truncated bitstream due to
the safe bitstream reader returning 0's at the end.

Fixes Bug 310.

CC:libav-stable@libav.org

Fixes bug 282.
Signed-off-by: NAnton Khirnov <anton@khirnov.net>