- Don't use LOG() macro in initdata() until the config struct has been

  allocated (uses config->debug).
- Initialise log_stream to stderr to catch errors before the config file
  is read.

diff --git a/Changes b/Changes
index ca2f6e0..e4ba742 100644 (file)
--- a/Changes
+++ b/Changes
@@ -1,4 +1,4 @@
-* Tue May 3 2005 Brendan O'Dea <bod@optusnet.com.au> 2.1.0
+* Thu May 5 2005 Brendan O'Dea <bod@optusnet.com.au> 2.1.0

 - Add IPv6 support from Jonathan McDowell.
 - Add CHAP support from Jordan Hrycaj (work in progress).
 - Sanity check that cluster_send_session is not called from a child
 - Add IPv6 support from Jonathan McDowell.
 - Add CHAP support from Jordan Hrycaj (work in progress).
 - Sanity check that cluster_send_session is not called from a child


@@ -48,6 +48,10 @@
 - Use cli_error() for error messages and help.
 - Add a Cisco-Avpair with intercept details to RADIUS Start/Stop
   records.
 - Use cli_error() for error messages and help.
 - Add a Cisco-Avpair with intercept details to RADIUS Start/Stop
   records.

+- Don't use LOG() macro in initdata() until the config struct has been
+  allocated (uses config->debug).
+- Initialise log_stream to stderr to catch errors before the config file
+  is read.

 
 * Fri Dec 17 2004 Brendan O'Dea <bod@optusnet.com.au> 2.0.13
 - Better cluster master collision resolution: keep a counter of state
 
 * Fri Dec 17 2004 Brendan O'Dea <bod@optusnet.com.au> 2.0.13
 - Better cluster master collision resolution: keep a counter of state

diff --git a/l2tpns.c b/l2tpns.c
index 6896717..1502f6b 100644 (file)
--- a/l2tpns.c
+++ b/l2tpns.c
@@ -4,7 +4,7 @@
 // Copyright (c) 2002 FireBrick (Andrews & Arnold Ltd / Watchfront Ltd) - GPL licenced
 // vim: sw=8 ts=8
 
 // Copyright (c) 2002 FireBrick (Andrews & Arnold Ltd / Watchfront Ltd) - GPL licenced
 // vim: sw=8 ts=8
 

-char const *cvs_id_l2tpns = "$Id: l2tpns.c,v 1.91 2005-04-27 13:53:15 bodea Exp $";
+char const *cvs_id_l2tpns = "$Id: l2tpns.c,v 1.92 2005-05-05 02:39:54 bodea Exp $";

 
 #include <arpa/inet.h>
 #include <assert.h>
 
 #include <arpa/inet.h>
 #include <assert.h>


@@ -55,23 +55,23 @@ char const *cvs_id_l2tpns = "$Id: l2tpns.c,v 1.91 2005-04-27 13:53:15 bodea Exp
 #endif /* BGP */
 
 // Globals
 #endif /* BGP */
 
 // Globals

-configt *config = NULL;                // all configuration
-int tunfd = -1;                        // tun interface file handle. (network device)
-int udpfd = -1;                        // UDP file handle
-int controlfd = -1;            // Control signal handle
-int clifd = -1;                        // Socket listening for CLI connections.
-int snoopfd = -1;              // UDP file handle for sending out intercept data
-int *radfds = NULL;            // RADIUS requests file handles
-int ifrfd = -1;                        // File descriptor for routing, etc
-int ifr6fd = -1;               // File descriptor for IPv6 routing, etc
-static int rand_fd = -1;       // Random data source
-time_t basetime = 0;           // base clock
-char hostname[1000] = "";      // us.
-static int tunidx;             // ifr_ifindex of tun device
-static int syslog_log = 0;     // are we logging to syslog
-static FILE *log_stream = NULL;        // file handle for direct logging (i.e. direct into file, not via syslog).
-extern int cluster_sockfd;     // Intra-cluster communications socket.
-uint32_t last_id = 0;          // Unique ID for radius accounting
+configt *config = NULL;                        // all configuration
+int tunfd = -1;                                // tun interface file handle. (network device)
+int udpfd = -1;                                // UDP file handle
+int controlfd = -1;                    // Control signal handle
+int clifd = -1;                                // Socket listening for CLI connections.
+int snoopfd = -1;                      // UDP file handle for sending out intercept data
+int *radfds = NULL;                    // RADIUS requests file handles
+int ifrfd = -1;                                // File descriptor for routing, etc
+int ifr6fd = -1;                       // File descriptor for IPv6 routing, etc
+static int rand_fd = -1;               // Random data source
+time_t basetime = 0;                   // base clock
+char hostname[1000] = "";              // us.
+static int tunidx;                     // ifr_ifindex of tun device
+static int syslog_log = 0;             // are we logging to syslog
+static FILE *log_stream = stderr;      // file handle for direct logging (i.e. direct into file, not via syslog).
+extern int cluster_sockfd;             // Intra-cluster communications socket.
+uint32_t last_id = 0;                  // Unique ID for radius accounting

 
 struct cli_session_actions *cli_session_actions = NULL;        // Pending session changes requested by CLI
 struct cli_tunnel_actions *cli_tunnel_actions = NULL;  // Pending tunnel changes required by CLI
 
 struct cli_session_actions *cli_session_actions = NULL;        // Pending session changes requested by CLI
 struct cli_tunnel_actions *cli_tunnel_actions = NULL;  // Pending tunnel changes required by CLI


@@ -3141,16 +3141,12 @@ static void initdata(int optdebug, char *optconfig)
 {
        int i;
 
 {
        int i;
 

-       if (!(_statistics = shared_malloc(sizeof(struct Tstats))))
-       {
-               LOG(0, 0, 0, "Error doing malloc for _statistics: %s\n", strerror(errno));
-               exit(1);
-       }

        if (!(config = shared_malloc(sizeof(configt))))
        {
        if (!(config = shared_malloc(sizeof(configt))))
        {

-               LOG(0, 0, 0, "Error doing malloc for configuration: %s\n", strerror(errno));
+               fprintf(stderr, "Error doing malloc for configuration: %s\n", strerror(errno));

                exit(1);
        }
                exit(1);
        }

+

        memset(config, 0, sizeof(configt));
        time(&config->start_time);
        strncpy(config->config_file, optconfig, strlen(optconfig));
        memset(config, 0, sizeof(configt));
        time(&config->start_time);
        strncpy(config->config_file, optconfig, strlen(optconfig));


@@ -3159,6 +3155,20 @@ static void initdata(int optdebug, char *optconfig)
        config->rl_rate = 28; // 28kbps
        strcpy(config->random_device, RANDOMDEVICE);
 
        config->rl_rate = 28; // 28kbps
        strcpy(config->random_device, RANDOMDEVICE);
 

+#ifdef RINGBUFFER
+       if (!(ringbuffer = shared_malloc(sizeof(struct Tringbuffer))))
+       {
+               LOG(0, 0, 0, "Error doing malloc for ringbuffer: %s\n", strerror(errno));
+               exit(1);
+       }
+       memset(ringbuffer, 0, sizeof(struct Tringbuffer));
+#endif
+
+       if (!(_statistics = shared_malloc(sizeof(struct Tstats))))
+       {
+               LOG(0, 0, 0, "Error doing malloc for _statistics: %s\n", strerror(errno));
+               exit(1);
+       }

        if (!(tunnel = shared_malloc(sizeof(tunnelt) * MAXTUNNEL)))
        {
                LOG(0, 0, 0, "Error doing malloc for tunnels: %s\n", strerror(errno));
        if (!(tunnel = shared_malloc(sizeof(tunnelt) * MAXTUNNEL)))
        {
                LOG(0, 0, 0, "Error doing malloc for tunnels: %s\n", strerror(errno));


@@ -3195,15 +3205,6 @@ static void initdata(int optdebug, char *optconfig)
        }
        memset(ip_filters, 0, sizeof(ip_filtert) * MAXFILTER);
 
        }
        memset(ip_filters, 0, sizeof(ip_filtert) * MAXFILTER);
 

-#ifdef RINGBUFFER
-       if (!(ringbuffer = shared_malloc(sizeof(struct Tringbuffer))))
-       {
-               LOG(0, 0, 0, "Error doing malloc for ringbuffer: %s\n", strerror(errno));
-               exit(1);
-       }
-       memset(ringbuffer, 0, sizeof(struct Tringbuffer));
-#endif
-

        if (!(cli_session_actions = shared_malloc(sizeof(struct cli_session_actions) * MAXSESSION)))
        {
                LOG(0, 0, 0, "Error doing malloc for cli session actions: %s\n", strerror(errno));
        if (!(cli_session_actions = shared_malloc(sizeof(struct cli_session_actions) * MAXSESSION)))
        {
                LOG(0, 0, 0, "Error doing malloc for cli session actions: %s\n", strerror(errno));


@@ -3771,9 +3772,11 @@ int main(int argc, char *argv[])
 
 static void sighup_handler(int sig)
 {
 
 static void sighup_handler(int sig)
 {

-       if (log_stream && log_stream != stderr)
+       if (log_stream)

        {
        {

-               fclose(log_stream);
+               if (log_stream != stderr)
+                       fclose(log_stream);
+

                log_stream = NULL;
        }
 
                log_stream = NULL;
        }
 


@@ -3895,7 +3898,9 @@ static void update_config()
        syslog_log = 0;
        if (log_stream)
        {
        syslog_log = 0;
        if (log_stream)
        {

-               fclose(log_stream);
+               if (log_stream != stderr)
+                       fclose(log_stream);
+

                log_stream = NULL;
        }
 
                log_stream = NULL;
        }
 

diff --git a/l2tpns.spec b/l2tpns.spec
index b9b9955..4c414b6 100644 (file)
--- a/l2tpns.spec
+++ b/l2tpns.spec
@@ -43,5 +43,5 @@ rm -rf %{buildroot}
 %attr(644,root,root) /usr/share/man/man[58]/*
 
 %changelog
 %attr(644,root,root) /usr/share/man/man[58]/*
 
 %changelog

-* Tue May 3 2005 Brendan O'Dea <bod@optusnet.com.au> 2.1.0-1
+* Thu May 5 2005 Brendan O'Dea <bod@optusnet.com.au> 2.1.0-1

 - 2.1.0 release, see /usr/share/doc/l2tpns-2.1.0/Changes
 - 2.1.0 release, see /usr/share/doc/l2tpns-2.1.0/Changes