[krbdev.mit.edu #7677] AutoReply: 1.10.5 & 1.10.6 kpropd patch

[Richard Basch via RT]

I added the patch to github:

https://github.com/rbasch/krb5/commit/cc4bf70b59d34ff76a329cdddc7ee03f9f5b55
d7


-----Original Message-----
From: krb5 [mailto:rt at krbdev.mit.edu] 
Sent: Saturday, July 06, 2013 4:30 PM
To: basch at mit.edu
Subject: [krbdev.mit.edu #7677] AutoReply: 1.10.5 & 1.10.6 kpropd patch 


Greetings,

This message has been automatically generated in response to the
creation of a trouble ticket regarding:
	"1.10.5 & 1.10.6 kpropd patch", 
a summary of which appears below.

There is no need to reply to this message right now.  Your ticket has been
assigned an ID of [krbdev.mit.edu #7677].

Please include the string:

         [krbdev.mit.edu #7677]

in the subject line of all future correspondence about this issue. To do so,

you may reply to this message.

                        Thank you,
                        

-------------------------------------------------------------------------
The fix ported back from 1.11.2 to 1.10.5 to resolve the potential issue
where a failed full resync might result in a database still reporting it is
current is flawed.  Attached is a patch which ensures if a full resync fails
the resulting database is not considered "current".

I have seen this bad database condition if kpropd logs an error similar to
"kpropd: Full resync, invalid return."

Sorry... I don't have my krb5 1.10 tree in github... it's only 10 additional
lines of code + 3 other lines for readability (comments, whitespace).	

-----Original Message-----
From: Richard Basch [mailto:probe at mail.bright-prospects.com] 
Sent: Saturday, July 06, 2013 12:33 AM
To: basch at mit.edu; richard.basch at gs.com
Subject: kpropd patch

--- src/slave/kpropd.c.orig	2013-04-17 20:40:18.000000000 -0400
+++ src/slave/kpropd.c	2013-07-06 00:27:45.135069434 -0400
@@ -931,6 +931,19 @@
             break;
         }
 
+	if (incr_ret->ret == UPDATE_FULL_RESYNC_NEEDED && frdone == 0) {
+	    /* Re-init ulog so we don't accidentally think we are current */
+	    if (log_ctx && log_ctx->iproprole) {
+		log_ctx->ulog->kdb_last_sno = 0;
+		log_ctx->ulog->kdb_last_time.seconds = 0;
+		log_ctx->ulog->kdb_last_time.useconds = 0;
+
+		log_ctx->ulog->kdb_first_sno = 0;
+		log_ctx->ulog->kdb_first_time.seconds = 0;
+		log_ctx->ulog->kdb_first_time.useconds = 0;
+	    }
+	}
+
         if (runonce == 1)
             goto done;