On 8/4/11 3:27 PM, Stefano Sabatini wrote:
On date Wednesday 2011-08-03 16:58:05 -0700, Ronald S. Bultje encoded:
Hi,

On Wed, Aug 3, 2011 at 4:44 PM, Stefano Sabatini
<[email protected]>  wrote:
+const static FormatEntry format_entries[PIX_FMT_NB] = {
[..]
  int sws_isSupportedInput(enum PixelFormat pix_fmt)
  {
-    return isSupportedIn(pix_fmt);
+    return format_entries[pix_fmt].is_supported_in;
  }

This is a public API function, no? It crashes for any newly added
(e.g. API change without recompiling lsws) PIX_FMT entry, and also for
invalid (e.g. negative) entries.

Right, updated with a check on pix_fmt.


Looks good, thank you =)

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to