Send_Message: Fix handling of "empty" targets

[ngircd-alex.git] / src / ngircd / irc.c

diff --git a/src/ngircd/irc.c b/src/ngircd/irc.c
index 15bb90f73ced63e4d7134d5f49b56fb893debd2a..5325b52aedaa7ef3cd470f9ef09a7a9c3bb2a2d7 100644 (file)
--- a/src/ngircd/irc.c
+++ b/src/ngircd/irc.c
@@ -563,7 +563,9 @@ Send_Message(CLIENT * Client, REQUEST * Req, int ForceType, bool SendErrors)
        currentTarget = strtok_r(currentTarget, ",", &strtok_last);
        ngt_UpperStr(Req->command);
 
-       while (true) {
+       /* Please note that "currentTarget" is NULL when the target contains
+        * the separator character only, e. g. "," or ",,,," etc.! */
+       while (currentTarget) {
                /* Make sure that there hasn't been such a target already: */
                targets[target_nr++] = currentTarget;
                for(i = 0; i < target_nr - 1; i++) {