diff options
| author | dequis <dx@dxzone.com.ar> | 2015-09-22 01:17:15 -0300 |
|---|---|---|
| committer | dequis <dx@dxzone.com.ar> | 2015-09-22 01:17:15 -0300 |
| commit | 99f074b0db2d723c3a1ddbdc5226f54f24b01989 (patch) | |
| tree | c229a706535599a59c4f29ea14db769859686465 /src/fe-common/irc/fe-irc-channels.c | |
| parent | bf36f71b99b18401dbc6105509b0285146b37286 (diff) | |
| download | irssi-99f074b0db2d723c3a1ddbdc5226f54f24b01989.zip | |
fe_channel_skip_prefix: fix return value (FALSE/NULL isn't valid)
The return value is a char*, and here it was false which is 0 which is
more or less the same as null.
That could have been a crash somewhere, the functions that call this
don't expect null ever.
Diffstat (limited to 'src/fe-common/irc/fe-irc-channels.c')
| -rw-r--r-- | src/fe-common/irc/fe-irc-channels.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/fe-common/irc/fe-irc-channels.c b/src/fe-common/irc/fe-irc-channels.c index d5f17bd0..a2737fc3 100644 --- a/src/fe-common/irc/fe-irc-channels.c +++ b/src/fe-common/irc/fe-irc-channels.c @@ -56,7 +56,7 @@ const char *fe_channel_skip_prefix(IRC_SERVER_REC *server, const char *target) /* Exit early if target doesn't name a channel */ if (server_ischannel(SERVER(server), target) == FALSE) - return FALSE; + return target; statusmsg = g_hash_table_lookup(server->isupport, "statusmsg"); |