[FFmpeg-soc] [PATCH] rtsp tunneling
Martin Storsjö
martin at martin.st
Fri Jun 4 11:39:14 CEST 2010
On Fri, 4 Jun 2010, Josh Allmann wrote:
> On 3 June 2010 07:34, Luca Barbato <lu_zero at gentoo.org> wrote:
> >
> > + if (!av_base64_encode(base64buf, sizeof(base64buf), buf,
> > strlen(buf))) {
> > + av_log(s, AV_LOG_ERROR, "Unable to base64 encode RTSP.\n");
> > + return;
> > + }
> >
> > currently this function assumes that it cannot fail, probably we should
> > propagate errors.
> >
>
> Done, added a fifth patch for this. I only check and propagate errors
> within ff_rtsp_send_cmd* functions, but I don't do anything about them
> when called from other ff_rtsp* functions.
Propagating errors generally is a very good idea. For this particular
case, the only thing that could fail with base64 is that the output buffer
isn't large enough, and we already make sure it's large enough. But I'm ok
with it either way.
// Martin
More information about the FFmpeg-soc
mailing list