On Fri, 25 Nov 2011, Diego Biurrun wrote:

On Fri, Nov 25, 2011 at 09:17:44AM +0200, Martin Storsjö wrote:
On Fri, 25 Nov 2011, Diego Biurrun wrote:
On Fri, Nov 25, 2011 at 01:16:39AM +0100, Janne Grunau wrote:
On Fri, Nov 25, 2011 at 12:49:42AM +0200, Martin Storsjö wrote:
This was removed erroneously in
046f081b46c8479820409cf8f530b988221bd15b. This define still is
necessary for getting MAP_ANONYMOUS defined on linux/glibc,
despite the define reshuffling done in that commit.

Without MAP_ANONYMOUS defined, the mprotect calls for setting the
generated mmx2 scaler code pages executable are left out, causing
crashes if that codepath is chosen.

This patch fixes scaling from 192x144 to 320x240 with
-sws_flags fast_bilinear, which crashes on linux at the
moment.
---

--- a/libswscale/utils.c
+++ b/libswscale/utils.c
@@ -18,6 +18,7 @@
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
 */

+#define _SVID_SOURCE //needed for MAP_ANONYMOUS
#include <inttypes.h>
#include <string.h>
#include <math.h>

ok.

can someone check if _DARWIN_C_SOURCE is requirred for MAP_ANON on
Mac OS X

Please don't push this right away until the issue is settled properly.
There was a similar patch floating around, possibly only for some OSes.
I'll look into it tomorrow.

Do you consider it settled properly after checking on OS X, or is
there anything else still to be checked?

Ping me over the weekend, there are still some outstanding issues I
want to check.

Pushed now after agreement on irc, Diego will follow up later if there's anything else to change.

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

Reply via email to