[Redbutton-devel] SF.net SVN: redbutton: [146] redbutton-browser/trunk/MHEGEngine.c

[<ski...@us...>]

Revision: 146
          http://svn.sourceforge.net/redbutton/?rev=146&view=rev
Author:   skilvington
Date:     2006-09-20 02:00:19 -0700 (Wed, 20 Sep 2006)

Log Message:
-----------
don't try to access memory after we've free'd it

Modified Paths:
--------------
    redbutton-browser/trunk/MHEGEngine.c

Modified: redbutton-browser/trunk/MHEGEngine.c
===================================================================
--- redbutton-browser/trunk/MHEGEngine.c	2006-09-11 10:07:05 UTC (rev 145)
+++ redbutton-browser/trunk/MHEGEngine.c	2006-09-20 09:00:19 UTC (rev 146)
@@ -329,6 +329,10 @@
 	|| OctetString_cmp(&current_scene->rootClass.inst.ref.group_identifier, &scene_id) != 0)
 	{
 		verbose("TransitionTo: %s", ExternalReference_name(&ref->u.external_reference));
+		/* UK MHEG Profile says we don't need to support transition effects */
+		/* (remember that 'to' will get destroyed when we free the action queues, so use it now if you need it) */
+		if(to->have_transition_effect)
+			error("Transition effects not supported");
 		/* get the active app */
 		current_app = MHEGEngine_getActiveApplication();
 		/* check the new scene is available */
@@ -386,9 +390,6 @@
 			SceneClass_Preparation(current_scene);
 			SceneClass_Activation(current_scene);
 		}
-		/* UK MHEG Profile says we don't need to support transition effects */
-		if(to->have_transition_effect)
-			error("Transition effects not supported");
 	}
 
 	/* clean up */


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.