Module Name:    src
Committed By:   joerg
Date:           Thu Apr 12 10:30:17 UTC 2012

Modified Files:
        src/libexec/lfs_cleanerd: fdfs.c

Log Message:
Use the same loop order for freeing as for allocation. Simpler code and
easier to process for analyzers.


To generate a diff of this commit:
cvs rdiff -u -r1.8 -r1.9 src/libexec/lfs_cleanerd/fdfs.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/libexec/lfs_cleanerd/fdfs.c
diff -u src/libexec/lfs_cleanerd/fdfs.c:1.8 src/libexec/lfs_cleanerd/fdfs.c:1.9
--- src/libexec/lfs_cleanerd/fdfs.c:1.8	Fri Jul  1 02:48:48 2011
+++ src/libexec/lfs_cleanerd/fdfs.c	Thu Apr 12 10:30:17 2012
@@ -1,4 +1,4 @@
-/* $NetBSD: fdfs.c,v 1.8 2011/07/01 02:48:48 joerg Exp $	 */
+/* $NetBSD: fdfs.c,v 1.9 2012/04/12 10:30:17 joerg Exp $	 */
 
 /*-
  * Copyright (c) 2005 The NetBSD Foundation, Inc.
@@ -98,7 +98,7 @@ fd_vget(int fd, int bsize, int segsize, 
 	vp = (struct uvnode *) malloc(sizeof(*vp));
 	if (vp == NULL) {
 		if (fs->fd_bufp) {
-			for (i = nseg - 1; i >= 0; i--)
+			for (i = 0; i < nseg; i++)
 				free(fs->fd_bufp[i].buf);
 			free(fs->fd_bufp);
 		}

Reply via email to