'for (part == 0; ...)' is not the same as 'for(part = 0; ...)'.
authorkrw <krw@openbsd.org>
Fri, 2 May 2014 12:43:10 +0000 (12:43 +0000)
committerkrw <krw@openbsd.org>
Fri, 2 May 2014 12:43:10 +0000 (12:43 +0000)
Should fix mysterious crashes when twiddling mount points. Too
much Marrakesh sun.

Pointed out by Owain G. Ainsworth. Thanks!

sbin/disklabel/editor.c

index e37d5a4..71fd9d0 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: editor.c,v 1.284 2014/04/21 08:19:38 krw Exp $        */
+/*     $OpenBSD: editor.c,v 1.285 2014/05/02 12:43:10 krw Exp $        */
 
 /*
  * Copyright (c) 1997-2000 Todd C. Miller <Todd.Miller@courtesan.com>
@@ -1900,7 +1900,7 @@ mpfree(char **mp)
        if (mp == NULL)
                return;
        
-       for (part == 0; part < MAXPARTITIONS; part++)
+       for (part = 0; part < MAXPARTITIONS; part++)
                free(mp[part]);
        
        free(mp);