repo: ngircd
action: commit
revision: 
path_from: 
revision_from: cd38e9580e428103cef39e2e29f62472b2f336d3:
path_to: 
revision_to: 
git.thebackupbox.net
ngircd
git clone git://git.thebackupbox.net/ngircd
commit cd38e9580e428103cef39e2e29f62472b2f336d3
Author: Federico G. Schwindt 
Date:   Sun Aug 11 20:57:11 2013 +0100

    Fix 2 bugs introduced in 74514b8c23

    Channel_Modes() and Channel_Topic() return static memory that is zeroed at
    creation time. Fix checking for empty modes and topic so they're propagated
    correctly.

diff --git a/src/ngircd/irc-channel.c b/src/ngircd/irc-channel.c
index 17ecbf9308c9c17b455b81f1b6d9ea5e55c0c96e..
index ..0978a19dfb30f9afcc1c2f937798a44be6f82a89 100644
--- a/src/ngircd/irc-channel.c
+++ b/src/ngircd/irc-channel.c
@@ -688,7 +688,7 @@ IRC_CHANINFO( CLIENT *Client, REQUEST *Req )
 		return CONNECTED;

 	if (Req->argv[1][0] == '+') {
-		if (!Channel_Modes(chan)) {
+		if (!*Channel_Modes(chan)) {
 			/* OK, this channel doesn't have modes yet,
 			 * set the received ones: */
 			Channel_SetModes(chan, &Req->argv[1][1]);
@@ -727,7 +727,7 @@ IRC_CHANINFO( CLIENT *Client, REQUEST *Req )

 	if (arg_topic > 0) {
 		/* We got a topic */
-		if (!Channel_Topic( chan ) && Req->argv[arg_topic][0]) {
+		if (!*Channel_Topic(chan) && Req->argv[arg_topic][0]) {
 			/* OK, there is no topic jet */
 			Channel_SetTopic(chan, Client, Req->argv[arg_topic]);
 			IRC_WriteStrChannelPrefix(Client, chan, from, false,

-----END OF PAGE-----