Skip site navigation (1)Skip section navigation (2) 
Date:      Sat, 8 Mar 2008 18:34:17 +0000 (UTC)
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   cvs commit: src/sys/netipsec key.c
Message-ID:  <200803081834.m28IYHB6035877@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help 

bz          2008-03-08 18:34:17 UTC

  FreeBSD src repository

  Modified files:        (Branch: RELENG_6)
    sys/netipsec         key.c 
  Log:
  MFC rev. 1.25
  
    Add a missing return so that we drop out in case of an error and
    do not continue with a NULL pointer. [1]
  
    While here change the return of the error handling code path above.
    I cannot see why we should always return 0 there. Neither does KAME
    nor do we in here for the similar check in all the other functions.
  
    Found with:     Coverity Prevent(tm) [1]
    CID:            2521
  
  Revision  Changes    Path
  1.20.2.3  +2 -3      src/sys/netipsec/key.c

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