From owner-cvs-lib  Tue Mar 19 11:27:13 1996
Return-Path: owner-cvs-lib
Received: (from root@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id LAA23857
          for cvs-lib-outgoing; Tue, 19 Mar 1996 11:27:13 -0800 (PST)
Received: (from wpaul@localhost)
          by freefall.freebsd.org (8.7.3/8.7.3) id LAA23837
          Tue, 19 Mar 1996 11:27:06 -0800 (PST)
Date: Tue, 19 Mar 1996 11:27:06 -0800 (PST)
From: Bill Paul <wpaul>
Message-Id: <199603191927.LAA23837@freefall.freebsd.org>
To: CVS-committers, cvs-all, cvs-lib
Subject: cvs commit:  src/lib/libc/yp yplib.c
Sender: owner-cvs-lib@FreeBSD.ORG
X-Loop: FreeBSD.org
Precedence: bulk

wpaul       96/03/19 11:27:05

  Modified:    lib/libc/yp  yplib.c
  Log:
  Fix yp_all() so that it doesn't bogusly return YP_NOMORE at the end
  of a successful map retrieval. (This has to do with a previous change
  to xdr_ypresp_all_seq() and ypxfr_get_map(); originally, yp_all()
  would look for a return value of YP_FALSE to signal success, but now
  it should be looking for YP_NOMORE. It should not be passing YP_NOMORE
  back up to the caller though.)
  
  Noticed by:  <aagero@aage.priv.no>
  
  There is also another small bug here, which is that the call to
  xdr_free() that happens immediately after the clnt_call() in yp_all()
  clobbers the return status value. I've worked around this for now,
  but I think the xdr_free() is actually bogus and should be removed.
  I want to check some more before I do that though.
  
  Revision  Changes    Path
  1.16      +5 -4      src/lib/libc/yp/yplib.c