commit 2f20f6e6b2cc6f26794678d1db16659c484a001c

Author: Panu Matilainen <pmatilai@redhat.com>

Date:   Wed Mar 21 09:07:21 2012 +0200

    Transaction element file info can be legally NULL in some cases

    - rpmteClose() will wipe out the file info to free memory, we only

      should care whether we failed to (re)load the file info. This

      thinko in commit 06a2f1269b035a3a76464149834f2a5a8c4e89f2

      broke %posttrans scriptlets (and without commit

      274dbf557d1cac90f7c278f9b6d6af05997d92df, %pretrans in other

      circumstances), whoopsie *blush*. Now, off to write a test-case

      for our scriptlet behavior...

diff --git a/lib/rpmte.c b/lib/rpmte.c

index 65e1e76..65d65af 100644

--- a/lib/rpmte.c

+++ b/lib/rpmte.c

@@ -693,11 +693,12 @@ static int rpmteOpen(rpmte te, int reload_fi)

     }

     if (h != NULL) {

 	if (reload_fi) {

+	    /* This can fail if we get a different, bad header from callback */

 	    te->fi = getFI(te, h);

+	    rc = (te->fi != NULL);

+	} else {

+	    rc = 1;

 	}

-

-	/* This can fail if we get a different, bad header from callback */

-	rc = (te->fi != NULL);

 	rpmteSetHeader(te, h);

 	headerFree(h);