Fixed stupid linked list bug.

icculus · Feb 3, 2009 · 8095057 · 8095057
1 parent 4e8ab34
commit 8095057
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 14 deletions.
diff --git a/mojoshader.c b/mojoshader.c
@@ -396,19 +396,20 @@ static void failf(Context *ctx, const char *fmt, ...)
         error->error.error = failstr;
         error->error.filename = NULL;  // no filename at this level.
         error->error.error_position = error_position;
+        error->next = NULL;
 
         ErrorList *prev = NULL;
-        error->next = ctx->errors;
-        while (error->next != NULL)
+        ErrorList *item = ctx->errors;
+        while (item != NULL)
         {
-            prev = error->next;
-            error->next = error->next->next;
+            prev = item;
+            item = error->next;
         } // while
 
-        if (prev != NULL)
-            prev->next = error;
-        else
+        if (prev == NULL)
             ctx->errors = error;
+        else
+            prev->next = error;
 
         ctx->error_count++;
     } // else

diff --git a/mojoshader_assembler.c b/mojoshader_assembler.c
@@ -145,19 +145,20 @@ static void failf(Context *ctx, const char *fmt, ...)
         error->error.error = failstr;
         error->error.filename = fname ? StrDup(ctx, fname) : NULL;
         error->error.error_position = error_position;
+        error->next = NULL;
 
         ErrorList *prev = NULL;
-        error->next = ctx->errors;
-        while (error->next != NULL)
+        ErrorList *item = ctx->errors;
+        while (item != NULL)
         {
-            prev = error->next;
-            error->next = error->next->next;
+            prev = item;
+            item = error->next;
         } // while
 
-        if (prev != NULL)
-            prev->next = error;
-        else
+        if (prev == NULL)
             ctx->errors = error;
+        else
+            prev->next = error;
 
         ctx->error_count++;
     } // else