RPM Community Forums

Mailing List Message of <rpm-cvs>

[CVS] RPM: rpm-5_1: rpm/ CHANGES rpm/lib/ manifest.c

From: Jeff Johnson <jbj@rpm5.org>
Date: Sun 07 Sep 2008 - 00:29:50 CEST
Message-Id: <20080906222950.6DB108F6@rpm5.org>
  RPM Package Manager, CVS Repository
  http://rpm5.org/cvs/
  ____________________________________________________________________________

  Server: rpm5.org                         Name:   Jeff Johnson
  Root:   /v/rpm/cvs                       Email:  jbj@rpm5.org
  Module: rpm                              Date:   07-Sep-2008 00:29:50
  Branch: rpm-5_1                          Handle: 2008090622294801

  Modified files:           (Branch: rpm-5_1)
    rpm                     CHANGES
    rpm/lib                 manifest.c

  Log:
    - jbj: display error msg for lusers attempting "rpm -qp /bin"
    (#461362).

  Summary:
    Revision    Changes     Path
    1.2288.2.107+1  -0      rpm/CHANGES
    2.22.4.1    +5  -1      rpm/lib/manifest.c
  ____________________________________________________________________________

  patch -p0 <<'@@ .'
  Index: rpm/CHANGES
  ============================================================================
  $ cvs diff -u -r1.2288.2.106 -r1.2288.2.107 CHANGES
  --- rpm/CHANGES	6 Sep 2008 19:00:37 -0000	1.2288.2.106
  +++ rpm/CHANGES	6 Sep 2008 22:29:48 -0000	1.2288.2.107
  @@ -1,4 +1,5 @@
   5.1.4 -> 5.1.5:
  +    - jbj: display error msg for lusers attempting "rpm -qp /bin" (#461362).
       - jbj: WR: remove version from config paths.
       - jbj: lua: fix: don't close dup'd stdout running lua scripts.
       - jbj: macosx: avoid multiple definitions of _program_name.
  @@ .
  patch -p0 <<'@@ .'
  Index: rpm/lib/manifest.c
  ============================================================================
  $ cvs diff -u -r2.22 -r2.22.4.1 manifest.c
  --- rpm/lib/manifest.c	19 Nov 2007 18:57:54 -0000	2.22
  +++ rpm/lib/manifest.c	6 Sep 2008 22:29:49 -0000	2.22.4.1
  @@ -95,7 +95,9 @@
   	/* Read next line. */
   	s = fgets(line, sizeof(line) - 1, f);
   	if (s == NULL) {
  -	    /* XXX Ferror check needed */
  +	    if (Ferror(xfd))
  +		rpmlog(RPMLOG_ERR, _("reading %s manifest failed: %s\n"),
  +			fdGetOPath(xfd), Fstrerror(xfd));
   	    break;
   	}
   
  @@ -119,6 +121,8 @@
   
   	/* Insure that file contains only ASCII */
   	if (*s < 32) {
  +	    rpmlog(RPMLOG_ERR, _("reading %s manifest, non-printable characters found\n"),
  +			fdGetOPath(xfd), Fstrerror(xfd));
   	    rpmrc = RPMRC_FAIL;	/* XXX reject non-printable manifests. */
   	    goto exit;
   	}
  @@ .
Received on Sun Sep 7 00:29:50 2008
Driven by Jeff Johnson and the RPM project team.
Hosted by OpenPKG and Ralf S. Engelschall.
Powered by FreeBSD and OpenPKG.