commit - fed22184c57be6959b1906138ce83f5476ec2efa
commit + 4d519cfdbf57344aeaa3610a302bf4c654c058f6
blob - 657fe78139ec11c6d572ca98d6336b14890de407
blob + 3282a8d0460533301ca0c4f58675548fa35db833
--- src/ngircd/channel.c
+++ src/ngircd/channel.c
@@ -136,7 +136,7 @@ Channel_InitPredefined( void )
 			Channel_SetTopic(new_chan, NULL, conf_chan->topic);
 
 		/* Evaluate modes strings with fake requests */
-		if(conf_chan->modes_num) {
+		if (conf_chan->modes_num) {
 			/* Prepare fake request structure */
 			strlcpy(name, conf_chan->name, sizeof(name));
 			Log(LOG_INFO, "Evaluating predefined channel modes for \"%s\".", name);
@@ -154,16 +154,31 @@ Channel_InitPredefined( void )
 					Req.argv[Req.argc++] = c;
 					c = strtok(0, " ");
 				}
-				if(Req.argc > 1)
+
+				if (Req.argc > 1) {
+					/* Handling of legacy "Key" and "MaxUsers" settings:
+					 * Enforce setting the respective mode(s), to support
+					 * the legacy "Mode = kl" notation, which was valid but
+					 * is an invalid MODE string: key and limit are missing!
+					 * So set them manually when "k" or "l" are detected in
+					 * the first MODE parameter ... */
+					if (Req.argc > 1 && strchr(Req.argv[1], 'k')) {
+						Channel_SetKey(new_chan, conf_chan->key);
+						Channel_ModeAdd(new_chan, 'k');
+					}
+					if (strchr(Req.argv[1], 'l')) {
+						Channel_SetMaxUsers(new_chan, conf_chan->maxusers);
+						Channel_ModeAdd(new_chan, 'l');
+					}
+
 					IRC_MODE(Client_ThisServer(), &Req);
+				}
 
 				/* Original channel modes srings are no longer needed */
 				free(conf_chan->modes[n]);
 			}
 		}
 
-		Channel_SetKey(new_chan, conf_chan->key);
-		Channel_SetMaxUsers(new_chan, conf_chan->maxusers);
 		Set_KeyFile(new_chan, conf_chan->keyfile);
 
 		Log(LOG_INFO,
blob - b5226a7d530ad272aceb8efc7943f2b5e95d9fc3
blob + bdf95e0935b28ed90a1322d9f75d1f0dd0d4148b
--- src/testsuite/server-login-test.e
+++ src/testsuite/server-login-test.e
@@ -45,7 +45,7 @@ expect {
 }
 expect {
 	timeout { exit 1 }
-	":ngircd.test.server CHANINFO #FullKeyed +Plk Secret 0 :"
+	":ngircd.test.server CHANINFO #FullKeyed +Pkl Secret 0 :"
 }
 expect {
 	timeout { exit 1 }