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); }