[FFmpeg-cvslog] r20565 - trunk/libavcodec/w32thread.c

[reimar]

Author: reimar
Date: Fri Nov 20 22:08:26 2009
New Revision: 20565

Log:
In win32 thread implementation do not access jobnr if the thread is asked
to terminate, jobnr it does not point to a valid location in that case.

Modified:
   trunk/libavcodec/w32thread.c

Modified: trunk/libavcodec/w32thread.c
==============================================================================
--- trunk/libavcodec/w32thread.c	Fri Nov 20 21:02:49 2009	(r20564)
+++ trunk/libavcodec/w32thread.c	Fri Nov 20 22:08:26 2009	(r20565)
@@ -48,16 +48,17 @@ static unsigned WINAPI attribute_align_a
         int ret, jobnr;
 //printf("thread_func %X enter wait\n", (int)v); fflush(stdout);
         WaitForSingleObject(c->work_sem, INFINITE);
+        // avoid trying to access jobnr if we should quit
+        if (!c->func && !c->func2)
+            break;
         WaitForSingleObject(c->job_sem, INFINITE);
         jobnr = (*c->jobnr)++;
         ReleaseSemaphore(c->job_sem, 1, 0);
 //printf("thread_func %X after wait (func=%X)\n", (int)v, (int)c->func); fflush(stdout);
         if(c->func)
             ret= c->func(c->avctx, (uint8_t *)c->arg + jobnr*c->argsize);
-        else if (c->func2)
-            ret= c->func2(c->avctx, c->arg, jobnr, c->threadnr);
         else
-            return 0;
+            ret= c->func2(c->avctx, c->arg, jobnr, c->threadnr);
         if (c->ret)
             c->ret[jobnr] = ret;
 //printf("thread_func %X signal complete\n", (int)v); fflush(stdout);