From: Mark Wooding <mdw@distorted.org.uk>
Date: Tue, 17 Feb 2015 21:20:12 +0000 (+0000)
Subject: server/keymgmt.c (km_samealgsp): Make sure the bulk transforms match.
X-Git-Tag: 1.0.0pre16.1~1
X-Git-Url: https://git.distorted.org.uk/~mdw/tripe/commitdiff_plain/41498fe453c0ccb8ac9cb3c47ac757bc5ab5be7a

server/keymgmt.c (km_samealgsp): Make sure the bulk transforms match.

Getting this wrong would cause a communication failure which is
annoyingly difficult to debug.
---

diff --git a/server/keymgmt.c b/server/keymgmt.c
index 17325dce..61cba579 100644
--- a/server/keymgmt.c
+++ b/server/keymgmt.c
@@ -374,6 +374,7 @@ int km_samealgsp(const kdata *kdx, const kdata *kdy)
   const algswitch *a = &kdx->algs, *aa = &kdy->algs;
 
   return (group_samep(kdx->g, kdy->g) &&
+	  a->bulk == aa->bulk &&
 	  a->c == aa->c && a->b == aa->b &&
 	  a->mgf == aa->mgf && a->h == aa->h &&
 	  a->m == aa->m && a->tagsz == aa->tagsz);