Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 25 Feb 2011 23:05:35 +0000 (UTC)
From:      Ed Schouten <ed@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r219045 - head/lib/libc/gen
Message-ID:  <201102252305.p1PN5Zf4016960@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ed
Date: Fri Feb 25 23:05:35 2011
New Revision: 219045
URL: http://svn.freebsd.org/changeset/base/219045

Log:
  Fix style(9) issues in pututxline(3).
  
  Also, make sure to initialize the `ret' variable properly.
  
  Reported by:	Cedric Jonas <cedric c84 eu>
  Patch by:	Garrett Cooper <yanegomi gmail com>

Modified:
  head/lib/libc/gen/pututxline.c

Modified: head/lib/libc/gen/pututxline.c
==============================================================================
--- head/lib/libc/gen/pututxline.c	Fri Feb 25 22:14:12 2011	(r219044)
+++ head/lib/libc/gen/pututxline.c	Fri Feb 25 23:05:35 2011	(r219045)
@@ -43,8 +43,8 @@ __FBSDID("$FreeBSD$");
 static FILE *
 futx_open(const char *file)
 {
-	struct stat sb;
 	FILE *fp;
+	struct stat sb;
 	int fd;
 
 	fd = _open(file, O_CREAT|O_RDWR|O_EXLOCK, 0644);
@@ -69,11 +69,14 @@ futx_open(const char *file)
 static int
 utx_active_add(const struct futx *fu)
 {
-	struct futx fe;
 	FILE *fp;
-	off_t partial = -1;
+	struct futx fe;
+	off_t partial;
 	int error, ret;
 
+	partial = -1;
+	ret = 0;
+
 	/*
 	 * Register user login sessions.  Overwrite entries of sessions
 	 * that have already been terminated.
@@ -132,8 +135,8 @@ exact:
 static int
 utx_active_remove(struct futx *fu)
 {
-	struct futx fe;
 	FILE *fp;
+	struct futx fe;
 	int error, ret;
 
 	/*
@@ -266,7 +269,9 @@ struct utmpx *
 pututxline(const struct utmpx *utmpx)
 {
 	struct futx fu;
-	int bad = 0;
+	int bad;
+
+	bad = 0;
 
 	utx_to_futx(utmpx, &fu);
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201102252305.p1PN5Zf4016960>