März 2014 - ngIRCd GIT Updates

Alexander Barton : Allow "DefaultUserModes" to set all possible modes
by alex＠arthur.barton.de 18 Mär '14

18 Mär '14

Module: ngircd.git Branch: master Commit: d314c75a37f0132a1a16658494d0f60a0c0083f2 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=d314c75a… Author: Alexander Barton <alex(a)barton.de> Date: Tue Mar 18 16:43:21 2014 +0100 Allow "DefaultUserModes" to set all possible modes Let IRC_MODE() detect that the "fake" MODE command originated on the local sever, which enables all modes to be settable using "DefaultUserModes" that can be set by regular MODE commands, including modes only settable by IRC Operators. --- doc/sample-ngircd.conf.tmpl | 4 ++-- man/ngircd.conf.5.tmpl | 4 ++-- src/ngircd/irc-mode.c | 7 +++++++ src/ngircd/login.c | 2 +- 4 files changed, 12 insertions(+), 5 deletions(-) diff --git a/doc/sample-ngircd.conf.tmpl b/doc/sample-ngircd.conf.tmpl index ec425bd..1d07822 100644 --- a/doc/sample-ngircd.conf.tmpl +++ b/doc/sample-ngircd.conf.tmpl @@ -172,8 +172,8 @@ ;ConnectIPv4 = yes # Default user mode(s) to set on new local clients. Please note that - # only modes can be set that the client could set on itself, you can't - # set "a" (away) or "o" (IRC Op), for example! Default: none. + # only modes can be set that the client could set using regular MODE + # commands, you can't set "a" (away) for example! Default: none. ;DefaultUserModes = i # Do DNS lookups when a client connects to the server. diff --git a/man/ngircd.conf.5.tmpl b/man/ngircd.conf.5.tmpl index 208b461..9b2ed08 100644 --- a/man/ngircd.conf.5.tmpl +++ b/man/ngircd.conf.5.tmpl @@ -267,8 +267,8 @@ Default: yes. .TP \fBDefaultUserModes\fR (string) Default user mode(s) to set on new local clients. Please note that only modes -can be set that the client could set on itself, you can't set "a" (away) or -"o" (IRC Op), for example! +can be set that the client could set using regular MODE commands, you can't +set "a" (away) for example! Default: none. .TP \fBDNS\fR (boolean) diff --git a/src/ngircd/irc-mode.c b/src/ngircd/irc-mode.c index fe98121..79ab2eb 100644 --- a/src/ngircd/irc-mode.c +++ b/src/ngircd/irc-mode.c @@ -68,6 +68,13 @@ IRC_MODE( CLIENT *Client, REQUEST *Req ) _IRC_GET_SENDER_OR_RETURN_(origin, Req, Client) + /* Test for "fake" MODE commands injected by this local instance, + * for example when handling the "DefaultUserModes" settings. + * This doesn't harm real commands, because prefixes of regular + * clients are checked in Validate_Prefix() and can't be faked. */ + if (Req->prefix && Client_Search(Req->prefix) == Client_ThisServer()) + Client = Client_Search(Req->prefix); + /* Channel or user mode? */ cl = NULL; chan = NULL; if (Client_IsValidNick(Req->argv[0])) diff --git a/src/ngircd/login.c b/src/ngircd/login.c index 7f427a3..64cc812 100644 --- a/src/ngircd/login.c +++ b/src/ngircd/login.c @@ -192,7 +192,7 @@ Login_User_PostAuth(CLIENT *Client) /* Set default user modes */ if (Conf_DefaultUserModes[0]) { snprintf(modes, sizeof(modes), "+%s", Conf_DefaultUserModes); - Req.prefix = Client_ThisServer(); + Req.prefix = Client_ID(Client_ThisServer()); Req.command = "MODE"; Req.argc = 2; Req.argv[0] = Client_ID(Client);

1 0

Alexander Barton : Spoofed prefixes: Really kill connection on non-server links
by alex＠arthur.barton.de 18 Mär '14

18 Mär '14

Module: ngircd.git Branch: master Commit: 5009ab3e8c5a6fe7db5c5ad1d3fdc8aecfc64b55 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=5009ab3e… Author: Alexander Barton <alex(a)barton.de> Date: Tue Mar 18 16:27:03 2014 +0100 Spoofed prefixes: Really kill connection on non-server links This fixes commit 6cbe1308 which only killed the connection when the spoofed prefix itself belonged to a non-server client. --- src/ngircd/parse.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ngircd/parse.c b/src/ngircd/parse.c index 94230c8..28dee6f 100644 --- a/src/ngircd/parse.c +++ b/src/ngircd/parse.c @@ -342,7 +342,7 @@ Validate_Prefix( CONN_ID Idx, REQUEST *Req, bool *Closed ) /* check if the client named in the prefix is expected * to come from that direction */ if (Client_NextHop(c) != client) { - if (Client_Type(c) != CLIENT_SERVER) { + if (Client_Type(client) != CLIENT_SERVER) { Log(LOG_ERR, "Spoofed prefix \"%s\" from \"%s\" (connection %d, command \"%s\"), closing connection!", Req->prefix, Client_ID(client), Idx, Req->command);

1 0

Alexander Barton : Implement user mode "F": "relaxed flood protection"
by alex＠arthur.barton.de 18 Mär '14

18 Mär '14

Module: ngircd.git Branch: master Commit: 5713c49c8480639f08f7fff82fb5a40e8566e1dc URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=5713c49c… Author: Alexander Barton <alex(a)barton.de> Date: Tue Mar 18 14:55:38 2014 +0100 Implement user mode "F": "relaxed flood protection" ngIRCd relaxes its flood protection for users having the user mode "F" set and allows them to rapidly send data to the daemon. This mode is only settable by IRC Operators and can cause problems in the network -- so be careful and only set it on "trusted" clients! User mode "F" is used by Bahamut for this purpose, for example, see <http://docs.dal.net/docs/modes.html#4.9>. --- doc/Modes.txt | 3 ++- src/ngircd/conn.c | 11 ++++++++++- src/ngircd/defines.h | 4 ++-- src/ngircd/irc-mode.c | 1 + 4 files changed, 15 insertions(+), 4 deletions(-) diff --git a/doc/Modes.txt b/doc/Modes.txt index aee7491..a931787 100644 --- a/doc/Modes.txt +++ b/doc/Modes.txt @@ -2,7 +2,7 @@ ngIRCd - Next Generation IRC Server http://ngircd.barton.de/ - (c)2001-2012 Alexander Barton and Contributors. + (c)2001-2014 Alexander Barton and Contributors. ngIRCd is free software and published under the terms of the GNU General Public License. @@ -26,6 +26,7 @@ channels he is using at the moment. B 20 User is flagged as a "bot". c 17 IRC operator wants to receive connect/disconnect NOTICEs. C 19 Only users that share a channel are allowed to send messages. + F 22 Relaxed flood protection (only settable by IRC Operators). i 0.0.1 User is "invisible". o 0.0.1 User is IRC operator. q 20 User is protected, can not be kicked from a channel. diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c index 77c8cd8..5c175df 100644 --- a/src/ngircd/conn.c +++ b/src/ngircd/conn.c @@ -1693,7 +1693,12 @@ Handle_Buffer(CONN_ID Idx) maxcmd *= 5; break; case CLIENT_SERVICE: - maxcmd = MAX_COMMANDS_SERVICE; break; + maxcmd = MAX_COMMANDS_SERVICE; + break; + case CLIENT_USER: + if (Client_HasMode(c, 'F')) + maxcmd = MAX_COMMANDS_SERVICE; + break; } for (i=0; i < maxcmd; i++) { @@ -2427,6 +2432,10 @@ Throttle_Connection(const CONN_ID Idx, CLIENT *Client, const int Reason, || Client_Type(Client) == CLIENT_SERVICE) return; + /* Don't throttle clients with user mode 'F' set */ + if (Client_HasMode(Client, 'F')) + return; + LogDebug("Throttling connection %d: code %d, value %d!", Idx, Reason, Value); Conn_SetPenalty(Idx, 1); diff --git a/src/ngircd/defines.h b/src/ngircd/defines.h index 361564f..4acdc47 100644 --- a/src/ngircd/defines.h +++ b/src/ngircd/defines.h @@ -1,6 +1,6 @@ /* * ngIRCd -- The Next Generation IRC Daemon - * Copyright (c)2001-2013 Alexander Barton (alex(a)barton.de) and Contributors. + * Copyright (c)2001-2014 Alexander Barton (alex(a)barton.de) and Contributors. * * This program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -173,7 +173,7 @@ #endif /** Supported user modes. */ -#define USERMODES "abBcCioqrRswx" +#define USERMODES "abBcCFioqrRswx" /** Supported channel modes. */ #define CHANMODES "abehiIklmMnoOPqQrRstvVz" diff --git a/src/ngircd/irc-mode.c b/src/ngircd/irc-mode.c index 6a67007..fe98121 100644 --- a/src/ngircd/irc-mode.c +++ b/src/ngircd/irc-mode.c @@ -222,6 +222,7 @@ Client_Mode( CLIENT *Client, REQUEST *Req, CLIENT *Origin, CLIENT *Target ) break; case 'c': /* Receive connect notices */ case 'q': /* KICK-protected user */ + case 'F': /* disable flood protection */ /* (only settable by IRC operators!) */ if (!set || Client_Type(Client) == CLIENT_SERVER || Client_HasMode(Origin, 'o'))

1 0

Alexander Barton : Handle "throttling" in a single function
by alex＠arthur.barton.de 18 Mär '14

18 Mär '14

Module: ngircd.git Branch: master Commit: 35f1db5f28245579efbbb06eed6eaf1a3d6d84bc URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=35f1db5f… Author: Alexander Barton <alex(a)barton.de> Date: Tue Mar 18 14:48:52 2014 +0100 Handle "throttling" in a single function ngIRCd uses "command throttling" and "bps throttling" (bytes per second). The states are detected in different functions, Conn_Handler() and Read_Request(), but handle the actual "throttling" in a common function: this enables us to guarantee consistent behavior and to disable throttling for special connections in only one place, eventually. --- src/ngircd/conn.c | 65 +++++++++++++++++++++++++++++++++++------------------ 1 file changed, 43 insertions(+), 22 deletions(-) diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c index 4dfe62f..77c8cd8 100644 --- a/src/ngircd/conn.c +++ b/src/ngircd/conn.c @@ -74,6 +74,9 @@ #define SD_LISTEN_FDS_START 3 /** systemd(8) socket activation offset */ +#define THROTTLE_CMDS 1 /** Throttling: max commands reached */ +#define THROTTLE_BPS 2 /** Throttling: max bps reached */ + static bool Handle_Write PARAMS(( CONN_ID Idx )); static bool Conn_Write PARAMS(( CONN_ID Idx, char *Data, size_t Len )); static int New_Connection PARAMS(( int Sock, bool IsSSL )); @@ -88,6 +91,8 @@ static void New_Server PARAMS(( int Server, ng_ipaddr_t *dest )); static void Simple_Message PARAMS(( int Sock, const char *Msg )); static int NewListener PARAMS(( const char *listen_addr, UINT16 Port )); static void Account_Connection PARAMS((void)); +static void Throttle_Connection PARAMS((const CONN_ID Idx, CLIENT *Client, + const int Reason, unsigned int Value)); static array My_Listeners; static array My_ConnArray; @@ -665,7 +670,7 @@ GLOBAL void Conn_Handler(void) { int i; - size_t wdatalen, bytes_processed; + size_t wdatalen; struct timeval tv; time_t t; @@ -684,17 +689,7 @@ Conn_Handler(void) if ((My_Connections[i].sock > NONE) && (array_bytes(&My_Connections[i].rbuf) > 0)) { /* ... and try to handle the received data */ - bytes_processed = Handle_Buffer(i); - /* if we processed data, and there might be - * more commands in the input buffer, do not - * try to read any more data now */ - if (bytes_processed && - array_bytes(&My_Connections[i].rbuf) > 2) { - LogDebug - ("Throttling connection %d: command limit reached!", - i); - Conn_SetPenalty(i, 1); - } + Handle_Buffer(i); } } @@ -1573,8 +1568,8 @@ Read_Request( CONN_ID Idx ) { /* Read buffer is full */ Log(LOG_ERR, - "Receive buffer space exhausted (connection %d): %d bytes", - Idx, array_bytes(&My_Connections[Idx].rbuf)); + "Receive buffer space exhausted (connection %d): %d/%d bytes", + Idx, array_bytes(&My_Connections[Idx].rbuf), READBUFFER_LEN); Conn_Close(Idx, "Receive buffer space exhausted", NULL, false); return; } @@ -1626,6 +1621,8 @@ Read_Request( CONN_ID Idx ) /* Update connection statistics */ My_Connections[Idx].bytes_in += len; + + /* Handle read buffer */ My_Connections[Idx].bps += Handle_Buffer(Idx); /* Make sure that there is still a valid client registered */ @@ -1651,14 +1648,8 @@ Read_Request( CONN_ID Idx ) } /* Look at the data in the (read-) buffer of this connection */ - if (Client_Type(c) != CLIENT_SERVER - && Client_Type(c) != CLIENT_UNKNOWNSERVER - && Client_Type(c) != CLIENT_SERVICE - && My_Connections[Idx].bps >= maxbps) { - LogDebug("Throttling connection %d: BPS exceeded! (%u >= %u)", - Idx, My_Connections[Idx].bps, maxbps); - Conn_SetPenalty(Idx, 1); - } + if (My_Connections[Idx].bps >= maxbps) + Throttle_Connection(Idx, c, THROTTLE_BPS, maxbps); } /* Read_Request */ /** @@ -1827,6 +1818,11 @@ Handle_Buffer(CONN_ID Idx) array_bytes(&My_Connections[Idx].rbuf)); #endif + /* If data has been processed but there is still data in the read + * buffer, the command limit triggered. Enforce the penalty time: */ + if (len_processed && array_bytes(&My_Connections[Idx].rbuf) > 2) + Throttle_Connection(Idx, c, THROTTLE_CMDS, maxcmd); + return len_processed; } /* Handle_Buffer */ @@ -2410,6 +2406,31 @@ Conn_GetFromProc(int fd) return NONE; } /* Conn_GetFromProc */ +/** + * Throttle a connection because of excessive usage. + * + * @param Reason The reason, see THROTTLE_xxx constants. + * @param Idx The connection index. + * @param Client The client of this connection. + * @param Seconds The time to delay this connection. + */ +static void +Throttle_Connection(const CONN_ID Idx, CLIENT *Client, const int Reason, + unsigned int Value) +{ + assert(Idx > NONE); + assert(Client != NULL); + + /* Never throttle servers or services, only interrupt processing */ + if (Client_Type(Client) == CLIENT_SERVER + || Client_Type(Client) == CLIENT_UNKNOWNSERVER + || Client_Type(Client) == CLIENT_SERVICE) + return; + + LogDebug("Throttling connection %d: code %d, value %d!", Idx, + Reason, Value); + Conn_SetPenalty(Idx, 1); +} #ifndef STRICT_RFC

1 0

Alexander Barton : Updated config.{guess|sub} to version 2014-03-13
by alex＠arthur.barton.de 18 Mär '14

18 Mär '14

Module: ngircd.git Branch: master Commit: 4c2acd55c6ecb8d9a6a1c6c5c86295fdd859820c URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=4c2acd55… Author: Alexander Barton <alex(a)barton.de> Date: Tue Mar 18 14:42:11 2014 +0100 Updated config.{guess|sub} to version 2014-03-13 --- config.guess | 329 +++++++++++++++++++--------------------------------------- config.sub | 89 ++++++++-------- 2 files changed, 154 insertions(+), 264 deletions(-) Diff: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commitdiff;h=4c2a…

1 0

Alexander Barton : Streamline DEBUG_ARRAY, DEBUG_BUFFER, DEBUG_IO, DEBUG_ZIP
by alex＠arthur.barton.de 17 Mär '14

17 Mär '14

Module: ngircd.git Branch: master Commit: f547981188a28844068e864dc1ed955ff173d216 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=f5479811… Author: Alexander Barton <alex(a)barton.de> Date: Mon Mar 17 23:52:08 2014 +0100 Streamline DEBUG_ARRAY, DEBUG_BUFFER, DEBUG_IO, DEBUG_ZIP Change all #define's to follow the form #define DEBUG_xxx {0|1} to disable (0, default) or enable (1) additional debug messages. And somewhat enhance some DEBUG_BUFFER messages. --- src/ngircd/array.c | 16 ++++++++-------- src/ngircd/conn-zip.c | 16 ++++++++-------- src/ngircd/conn.c | 17 ++++++++++------- src/ngircd/io.c | 8 ++++---- 4 files changed, 30 insertions(+), 27 deletions(-) diff --git a/src/ngircd/array.c b/src/ngircd/array.c index af66edd..4cc793f 100644 --- a/src/ngircd/array.c +++ b/src/ngircd/array.c @@ -14,6 +14,9 @@ * Functions to dynamically allocate arrays. */ +/* Additionan debug messages related to array handling: 0=off / 1=on */ +#define DEBUG_ARRAY 0 + #include "array.h" #include <assert.h> @@ -21,13 +24,10 @@ #include <string.h> #include <unistd.h> -#ifdef DEBUG_ARRAY +#if DEBUG_ARRAY # include "log.h" #endif -/* Enable more Debug messages in alloc / append / memmove code. */ -/* #define DEBUG_ARRAY */ - #define array_UNUSABLE(x) ( !(x)->mem ) @@ -67,7 +67,7 @@ array_alloc(array * a, size_t size, size_t pos) return NULL; if (a->allocated < alloc) { -#ifdef DEBUG_ARRAY +#if DEBUG_ARRAY Log(LOG_DEBUG, "array_alloc(): changing size from %u to %u bytes.", a->allocated, alloc); #endif @@ -168,7 +168,7 @@ array_catb(array * dest, const char *src, size_t len) assert(ptr != NULL); -#ifdef DEBUG_ARRAY +#if DEBUG_ARRAY Log(LOG_DEBUG, "array_catb(): appending %u bytes to array (now %u bytes in array).", len, tmp); @@ -248,7 +248,7 @@ void array_free(array * a) { assert(a != NULL); -#ifdef DEBUG_ARRAY +#if DEBUG_ARRAY Log(LOG_DEBUG, "array_free(): %u bytes free'd (%u bytes still used at time of free()).", a->allocated, a->used); @@ -314,7 +314,7 @@ array_moveleft(array * a, size_t membersize, size_t pos) if (!bytepos) return; /* nothing to do */ -#ifdef DEBUG_ARRAY +#if DEBUG_ARRAY Log(LOG_DEBUG, "array_moveleft(): %u bytes used in array, starting at position %u.", a->used, bytepos); diff --git a/src/ngircd/conn-zip.c b/src/ngircd/conn-zip.c index 5ebb2a0..e21dd34 100644 --- a/src/ngircd/conn-zip.c +++ b/src/ngircd/conn-zip.c @@ -9,6 +9,8 @@ * Please read the file COPYING, README and AUTHORS for more information. */ +#define CONN_MODULE + #include "portab.h" /** @@ -16,13 +18,11 @@ * Connection compression using ZLIB */ -#define CONN_MODULE +/* Additionan debug messages related to ZIP compression: 0=off / 1=on */ +#define DEBUG_ZIP 0 #ifdef ZLIB -/* enable more zlib related debug messages: */ -/* #define DEBUG_ZLIB */ - #include <assert.h> #include <string.h> #include <zlib.h> @@ -141,7 +141,7 @@ Zip_Flush( CONN_ID Idx ) out->next_out = zipbuf; out->avail_out = (uInt)sizeof zipbuf; -#ifdef DEBUG_ZIP +#if DEBUG_ZIP Log(LOG_DEBUG, "out->avail_in %d, out->avail_out %d", out->avail_in, out->avail_out); #endif @@ -164,7 +164,7 @@ Zip_Flush( CONN_ID Idx ) assert(out->avail_out <= WRITEBUFFER_SLINK_LEN); zipbuf_used = WRITEBUFFER_SLINK_LEN - out->avail_out; -#ifdef DEBUG_ZIP +#if DEBUG_ZIP Log(LOG_DEBUG, "zipbuf_used: %d", zipbuf_used); #endif if (!array_catb(&My_Connections[Idx].wbuf, @@ -216,7 +216,7 @@ Unzip_Buffer( CONN_ID Idx ) in->next_out = unzipbuf; in->avail_out = (uInt)sizeof unzipbuf; -#ifdef DEBUG_ZIP +#if DEBUG_ZIP Log(LOG_DEBUG, "in->avail_in %d, in->avail_out %d", in->avail_in, in->avail_out); #endif @@ -231,7 +231,7 @@ Unzip_Buffer( CONN_ID Idx ) assert(z_rdatalen >= in->avail_in); in_len = z_rdatalen - in->avail_in; unzipbuf_used = READBUFFER_LEN - in->avail_out; -#ifdef DEBUG_ZIP +#if DEBUG_ZIP Log(LOG_DEBUG, "unzipbuf_used: %d - %d = %d", READBUFFER_LEN, in->avail_out, unzipbuf_used); #endif diff --git a/src/ngircd/conn.c b/src/ngircd/conn.c index 4ed5862..4dfe62f 100644 --- a/src/ngircd/conn.c +++ b/src/ngircd/conn.c @@ -9,8 +9,6 @@ * Please read the file COPYING, README and AUTHORS for more information. */ -#undef DEBUG_BUFFER - #define CONN_MODULE #include "portab.h" @@ -20,6 +18,9 @@ * Connection management */ +/* Additionan debug messages related to buffer handling: 0=off / 1=on */ +#define DEBUG_BUFFER 0 + #include <assert.h> #ifdef PROTOTYPES # include <stdarg.h> @@ -1265,7 +1266,7 @@ Handle_Write( CONN_ID Idx ) return true; } -#ifdef DEBUG_BUFFER +#if DEBUG_BUFFER LogDebug ("Handle_Write() called for connection %d, %ld bytes pending ...", Idx, wdatalen); @@ -1798,10 +1799,6 @@ Handle_Buffer(CONN_ID Idx) return 0; /* error -> connection has been closed */ array_moveleft(&My_Connections[Idx].rbuf, 1, len); -#ifdef DEBUG_BUFFER - LogDebug("Connection %d: %d bytes left in read buffer.", - Idx, array_bytes(&My_Connections[Idx].rbuf)); -#endif #ifdef ZLIB if ((!old_z) && (My_Connections[Idx].options & CONN_ZIP) && (array_bytes(&My_Connections[Idx].rbuf) > 0)) { @@ -1824,6 +1821,12 @@ Handle_Buffer(CONN_ID Idx) } #endif } +#if DEBUG_BUFFER + LogDebug("Connection %d: Processed %ld commands (max=%ld), %ld bytes. %ld bytes left in read buffer.", + Idx, i, maxcmd, len_processed, + array_bytes(&My_Connections[Idx].rbuf)); +#endif + return len_processed; } /* Handle_Buffer */ diff --git a/src/ngircd/io.c b/src/ngircd/io.c index 779e749..037c4af 100644 --- a/src/ngircd/io.c +++ b/src/ngircd/io.c @@ -17,6 +17,9 @@ * I/O abstraction interface. */ +/* Extra debug messages in event add/delete/callback code: 0=off / 1=on */ +#define DEBUG_IO 0 + #include <assert.h> #include <string.h> #include <sys/types.h> @@ -28,9 +31,6 @@ #include "io.h" #include "log.h" -/* Enables extra debug messages in event add/delete/callback code. */ -/* #define DEBUG_IO */ - typedef struct { #ifdef PROTOTYPES void (*callback)(int, short); @@ -144,7 +144,7 @@ static array io_events; static void io_docallback PARAMS((int fd, short what)); -#ifdef DEBUG_IO +#if DEBUG_IO static void io_debug(const char *s, int fd, int what) {

1 0

Alexander Barton : conn.c: Code cleanup
by alex＠arthur.barton.de 17 Mär '14

17 Mär '14

Module: ngircd.git Branch: master Commit: b35f8916a5252182070d0e4502a540e81a3ced90 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=b35f8916… Author: Alexander Barton <alex(a)barton.de> Date: Mon Mar 17 22:59:58 2014 +0100 conn.c: Code cleanup Reformat and restructure some code, move all SSL related code into one single #ifdef block, for example. --- src/ngircd/conn.c | 431 ++++++++++++++++++++++++----------------------------- 1 file changed, 198 insertions(+), 233 deletions(-) Diff: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commitdiff;h=b35f…

1 0

Alexander Barton : Xcode: remove imp.h and exp.h from project file
by alex＠arthur.barton.de 17 Mär '14

17 Mär '14

Module: ngircd.git Branch: master Commit: 3d7aa9f90626bb16ef9db9c9c739ba748976448c URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=3d7aa9f9… Author: Alexander Barton <alex(a)barton.de> Date: Mon Mar 17 18:07:51 2014 +0100 Xcode: remove imp.h and exp.h from project file These files have been removed from ngIRCd in commit 259c314d, "Remove imp.h and exp.h header files". --- contrib/MacOSX/ngIRCd.xcodeproj/project.pbxproj | 4 ---- 1 file changed, 4 deletions(-) diff --git a/contrib/MacOSX/ngIRCd.xcodeproj/project.pbxproj b/contrib/MacOSX/ngIRCd.xcodeproj/project.pbxproj index babe5bc..41ddb52 100644 --- a/contrib/MacOSX/ngIRCd.xcodeproj/project.pbxproj +++ b/contrib/MacOSX/ngIRCd.xcodeproj/project.pbxproj @@ -138,8 +138,6 @@ FA322D0D0CEF74B1001761B3 /* resolve.h */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.h; path = resolve.h; sourceTree = "<group>"; }; FA322D100CEF74B1001761B3 /* ansi2knr.1 */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = text.man; path = ansi2knr.1; sourceTree = "<group>"; }; FA322D110CEF74B1001761B3 /* ansi2knr.c */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.c; path = ansi2knr.c; sourceTree = "<group>"; }; - FA322D120CEF74B1001761B3 /* exp.h */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.h; path = exp.h; sourceTree = "<group>"; }; - FA322D130CEF74B1001761B3 /* imp.h */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.h; path = imp.h; sourceTree = "<group>"; }; FA322D150CEF74B1001761B3 /* portab.h */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.h; path = portab.h; sourceTree = "<group>"; }; FA322D160CEF74B1001761B3 /* portabtest.c */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.c; path = portabtest.c; sourceTree = "<group>"; }; FA322D170CEF74B1001761B3 /* splint.h */ = {isa = PBXFileReference; fileEncoding = 5; lastKnownFileType = sourcecode.c.h; path = splint.h; sourceTree = "<group>"; }; @@ -423,8 +421,6 @@ FA18A64716CEE14900132F66 /* Makefile.ng */, FA322D100CEF74B1001761B3 /* ansi2knr.1 */, FA322D110CEF74B1001761B3 /* ansi2knr.c */, - FA322D120CEF74B1001761B3 /* exp.h */, - FA322D130CEF74B1001761B3 /* imp.h */, FA322D150CEF74B1001761B3 /* portab.h */, FA322D160CEF74B1001761B3 /* portabtest.c */, FA322D170CEF74B1001761B3 /* splint.h */,

1 0

Alexander Barton : Re-add #include's for header files of the C file itself
by alex＠arthur.barton.de 17 Mär '14

17 Mär '14

Module: ngircd.git Branch: master Commit: a534e71e8da9a7bab55e7e76358a60dcbd3e25d5 URL: http://ngircd.barton.de/cgi-bin/gitweb.cgi?p=ngircd.git&a=commit;h=a534e71e… Author: Alexander Barton <alex(a)barton.de> Date: Mon Mar 17 18:02:57 2014 +0100 Re-add #include's for header files of the C file itself This partially reverts commit b130b35f4, "Update #include's: remove unused and add missing ones", but fixes the following compiler and analyzer warnings of Apple Xcode 5: "Semantic issue: No previous prototype for function 'yyy'" --- src/ngircd/client-cap.c | 2 ++ src/ngircd/hash.c | 2 ++ src/ngircd/irc-cap.c | 2 ++ src/ngircd/irc-channel.c | 2 ++ src/ngircd/irc-encoding.c | 2 ++ src/ngircd/irc-login.c | 2 ++ src/ngircd/irc-metadata.c | 2 ++ src/ngircd/irc-mode.c | 2 ++ src/ngircd/irc-op.c | 2 ++ src/ngircd/irc-oper.c | 2 ++ src/ngircd/irc-server.c | 2 ++ src/ngircd/match.c | 2 ++ src/ngircd/numeric.c | 2 ++ src/ngircd/op.c | 2 ++ src/ngircd/resolve.c | 2 ++ src/ngircd/sighandlers.c | 2 ++ 16 files changed, 32 insertions(+) diff --git a/src/ngircd/client-cap.c b/src/ngircd/client-cap.c index e6a4eb2..e403da1 100644 --- a/src/ngircd/client-cap.c +++ b/src/ngircd/client-cap.c @@ -23,6 +23,8 @@ #include "conn.h" #include "log.h" +#include "client-cap.h" + GLOBAL int Client_Cap(CLIENT *Client) { diff --git a/src/ngircd/hash.c b/src/ngircd/hash.c index 7e5ad9d..cdac5e8 100644 --- a/src/ngircd/hash.c +++ b/src/ngircd/hash.c @@ -22,6 +22,8 @@ #include "defines.h" #include "tool.h" +#include "hash.h" + static UINT32 jenkins_hash PARAMS((UINT8 *k, UINT32 length, UINT32 initval)); /** diff --git a/src/ngircd/irc-cap.c b/src/ngircd/irc-cap.c index 61e47f8..bf4c9cd 100644 --- a/src/ngircd/irc-cap.c +++ b/src/ngircd/irc-cap.c @@ -29,6 +29,8 @@ #include "messages.h" #include "parse.h" +#include "irc-cap.h" + /* Local functions */ /** diff --git a/src/ngircd/irc-channel.c b/src/ngircd/irc-channel.c index 3ea8232..c142243 100644 --- a/src/ngircd/irc-channel.c +++ b/src/ngircd/irc-channel.c @@ -35,6 +35,8 @@ #include "irc-write.h" #include "conf.h" +#include "irc-channel.h" + /** * Part from all channels. * diff --git a/src/ngircd/irc-encoding.c b/src/ngircd/irc-encoding.c index 2c6ac59..7b7121b 100644 --- a/src/ngircd/irc-encoding.c +++ b/src/ngircd/irc-encoding.c @@ -29,6 +29,8 @@ #include "parse.h" #include "tool.h" +#include "irc-encoding.h" + /** * Handler for the IRC+ "CHARCONV" command. * diff --git a/src/ngircd/irc-login.c b/src/ngircd/irc-login.c index 609b187..af165cf 100644 --- a/src/ngircd/irc-login.c +++ b/src/ngircd/irc-login.c @@ -34,6 +34,8 @@ #include "irc-macros.h" #include "irc-write.h" +#include "irc-login.h" + static void Change_Nick PARAMS((CLIENT * Origin, CLIENT * Target, char *NewNick, bool InformClient)); diff --git a/src/ngircd/irc-metadata.c b/src/ngircd/irc-metadata.c index 32e0451..efb2201 100644 --- a/src/ngircd/irc-metadata.c +++ b/src/ngircd/irc-metadata.c @@ -29,6 +29,8 @@ #include "messages.h" #include "parse.h" +#include "irc-metadata.h" + /** * Handler for the IRC+ "METADATA" command. * diff --git a/src/ngircd/irc-mode.c b/src/ngircd/irc-mode.c index 12a7e30..6a67007 100644 --- a/src/ngircd/irc-mode.c +++ b/src/ngircd/irc-mode.c @@ -31,6 +31,8 @@ #include "messages.h" #include "conf.h" +#include "irc-mode.h" + static bool Client_Mode PARAMS((CLIENT *Client, REQUEST *Req, CLIENT *Origin, CLIENT *Target)); static bool Channel_Mode PARAMS((CLIENT *Client, REQUEST *Req, CLIENT *Origin, diff --git a/src/ngircd/irc-op.c b/src/ngircd/irc-op.c index 82a78eb..b75841a 100644 --- a/src/ngircd/irc-op.c +++ b/src/ngircd/irc-op.c @@ -28,6 +28,8 @@ #include "messages.h" #include "parse.h" +#include "irc-op.h" + /* Local functions */ static bool diff --git a/src/ngircd/irc-oper.c b/src/ngircd/irc-oper.c index f896098..1d69a6c 100644 --- a/src/ngircd/irc-oper.c +++ b/src/ngircd/irc-oper.c @@ -37,6 +37,8 @@ #include "messages.h" #include "op.h" +#include "irc-oper.h" + /** * Handle invalid received OPER command. * Log OPER attempt and send error message to client. diff --git a/src/ngircd/irc-server.c b/src/ngircd/irc-server.c index 5ac8810..92186af 100644 --- a/src/ngircd/irc-server.c +++ b/src/ngircd/irc-server.c @@ -35,6 +35,8 @@ #include "irc-write.h" #include "op.h" +#include "irc-server.h" + /** * Handler for the IRC "SERVER" command. * diff --git a/src/ngircd/match.c b/src/ngircd/match.c index 3acae90..93ddc0b 100644 --- a/src/ngircd/match.c +++ b/src/ngircd/match.c @@ -22,6 +22,8 @@ #include "defines.h" #include "tool.h" +#include "match.h" + /* * The pattern matching functions [Matche(), Matche_After_Star()] are based * on code of J. Kercheval. Version 1.1 has been released on 1991-03-12 as diff --git a/src/ngircd/numeric.c b/src/ngircd/numeric.c index 5217e14..da863e4 100644 --- a/src/ngircd/numeric.c +++ b/src/ngircd/numeric.c @@ -31,6 +31,8 @@ #include "log.h" #include "parse.h" +#include "numeric.h" + /** * Announce a channel and its users in the network. */ diff --git a/src/ngircd/op.c b/src/ngircd/op.c index 7b7befb..4a11355 100644 --- a/src/ngircd/op.c +++ b/src/ngircd/op.c @@ -26,6 +26,8 @@ #include "messages.h" #include "irc-write.h" +#include "op.h" + /** * Return and log a "no privileges" message. */ diff --git a/src/ngircd/resolve.c b/src/ngircd/resolve.c index 01e303e..be41ec8 100644 --- a/src/ngircd/resolve.c +++ b/src/ngircd/resolve.c @@ -40,6 +40,8 @@ #include "log.h" #include "ng_ipaddr.h" +#include "resolve.h" + static void Do_ResolveAddr PARAMS(( const ng_ipaddr_t *Addr, int Sock, int w_fd )); static void Do_ResolveName PARAMS(( const char *Host, int w_fd )); diff --git a/src/ngircd/sighandlers.c b/src/ngircd/sighandlers.c index e5345db..5cde24b 100644 --- a/src/ngircd/sighandlers.c +++ b/src/ngircd/sighandlers.c @@ -33,6 +33,8 @@ #include "log.h" #include "ngircd.h" +#include "sighandlers.h" + static int signalpipe[2]; static const int signals_catch[] = {

1 0

Alexander Barton : Update copyright notices of recently changed files
by alex＠arthur.barton.de 17 Mär '14

17 Mär '14

1 0