From owner-cvs-src-old@FreeBSD.ORG Thu Jun 17 19:17:48 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 11D99106568C for ; Thu, 17 Jun 2010 19:17:48 +0000 (UTC) (envelope-from pjd@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id DA7388FC16 for ; Thu, 17 Jun 2010 19:17:47 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id o5HJHlbF070955 for ; Thu, 17 Jun 2010 19:17:47 GMT (envelope-from pjd@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id o5HJHlt7070954 for cvs-src-old@freebsd.org; Thu, 17 Jun 2010 19:17:47 GMT (envelope-from pjd@repoman.freebsd.org) Message-Id: <201006171917.o5HJHlt7070954@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to pjd@repoman.freebsd.org using -f From: Pawel Jakub Dawidek Date: Thu, 17 Jun 2010 19:17:31 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_8 Subject: cvs commit: src/sbin/hastd ebuf.c hast_proto.c hastd.c metadata.c nv.c primary.c secondary.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Jun 2010 19:17:48 -0000 pjd 2010-06-17 19:17:31 UTC FreeBSD src repository Modified files: (Branch: RELENG_8) sbin/hastd ebuf.c hast_proto.c hastd.c metadata.c nv.c primary.c secondary.c Log: SVN rev 209263 on 2010-06-17 19:17:31Z by pjd MFC r209175,r209177,r209179,r209180,r209181,r209182,r209183,r209184,r209185: r209175: Eliminate dead code. Found by: Coverity Prevent CID: 5158 r209177: Remove macros that are not really needed. The idea was to have them in case we grow more descriptors, but I'll reconsider readding them once we get there. Passing (a = b) expression to FD_ISSET() is bad idea, as FD_ISSET() evaluates its argument twice. Found by: Coverity Prevent CID: 5243 r209179: Plug memory leaks. Found by: Coverity Prevent CID: 7052, 7053, 7054, 7055 r209180: Plug memory leak. Found by: Coverity Prevent CID: 7051 r209181: Plug memory leak. Found by: Coverity Prevent CID: 7056 r209182: Plug memory leak. Found by: Coverity Prevent CID: 7057 r209183: Initialize gctl_seq for synchronization requests. Reported by: hiroshi@soupacific.com Analysed by: Mikolaj Golub Tested by: hiroshi@soupacific.com, Mikolaj Golub r209184: Fix typos. r209185: Correct various log messages. Submitted by: Mikolaj Golub Revision Changes Path 1.1.2.3 +7 -7 src/sbin/hastd/ebuf.c 1.1.2.4 +1 -3 src/sbin/hastd/hast_proto.c 1.2.2.5 +14 -22 src/sbin/hastd/hastd.c 1.1.2.4 +10 -6 src/sbin/hastd/metadata.c 1.1.2.3 +2 -0 src/sbin/hastd/nv.c 1.3.2.5 +5 -1 src/sbin/hastd/primary.c 1.1.2.4 +2 -1 src/sbin/hastd/secondary.c