remove dead session fields

[l2tpns.git] / ppp.c

diff --git a/ppp.c b/ppp.c
index 074e00f..b954e32 100644 (file)
--- a/ppp.c
+++ b/ppp.c
@@ -1,6 +1,6 @@
 // L2TPNS PPP Stuff
 
 // L2TPNS PPP Stuff
 

-char const *cvs_id_ppp = "$Id: ppp.c,v 1.42 2005/01/25 04:19:06 bodea Exp $";
+char const *cvs_id_ppp = "$Id: ppp.c,v 1.46 2005/03/10 06:16:05 bodea Exp $";

 
 #include <stdio.h>
 #include <string.h>
 
 #include <stdio.h>
 #include <string.h>


@@ -28,8 +28,8 @@ static void initccp(tunnelidt t, sessionidt s);
 // Process PAP messages
 void processpap(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
 {
 // Process PAP messages
 void processpap(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
 {

-       char user[129];
-       char pass[129];
+       char user[MAXUSER];
+       char pass[MAXPASS];

        uint16_t hl;
 
        CSTAT(processpap);
        uint16_t hl;
 
        CSTAT(processpap);


@@ -60,13 +60,18 @@ void processpap(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
        {
                uint8_t *b = p;
                b += 4;
        {
                uint8_t *b = p;
                b += 4;

+               user[0] = pass[0] = 0;

                if (*b && *b < sizeof(user))
                if (*b && *b < sizeof(user))

+               {

                        memcpy(user, b + 1, *b);
                        memcpy(user, b + 1, *b);

-               user[*b] = 0;
-               b += 1 + *b;
-               if (*b && *b < sizeof(pass))
-                       memcpy(pass, b + 1, *b);
-               pass[*b] = 0;
+                       user[*b] = 0;
+                       b += 1 + *b;
+                       if (*b && *b < sizeof(pass))
+                       {
+                               memcpy(pass, b + 1, *b);
+                               pass[*b] = 0;
+                       }
+               }

                LOG(3, s, t, "PAP login %s/%s\n", user, pass);
        }
        if (session[s].ip || !session[s].radius)
                LOG(3, s, t, "PAP login %s/%s\n", user, pass);
        }
        if (session[s].ip || !session[s].radius)


@@ -483,11 +488,11 @@ void processlcp(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
                q = makeppp(b, sizeof(b),  p, l, t, s, PPPLCP);
                if (!q) return;
                tunnelsend(b, l + (q - b), t); // send it
                q = makeppp(b, sizeof(b),  p, l, t, s, PPPLCP);
                if (!q) return;
                tunnelsend(b, l + (q - b), t); // send it

-               sessionshutdown(s, "Remote end closed connection.");
+               sessionshutdown(s, "Remote end closed connection.", 3, 0);

        }
        else if (*p == TerminateAck)
        {
        }
        else if (*p == TerminateAck)
        {

-               sessionshutdown(s, "Connection closed.");
+               sessionshutdown(s, "Connection closed.", 3, 0);

        }
        else if (*p == ProtocolRej)
        {
        }
        else if (*p == ProtocolRej)
        {


@@ -852,15 +857,19 @@ void processipin(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
 
        // Add on the tun header
        p -= 4;
 
        // Add on the tun header
        p -= 4;

-       *(uint32_t *) p = htonl(0x00000800);
+       *(uint32_t *) p = htonl(PKTIP);

        l += 4;
 
        l += 4;
 

-       if (session[s].tbf_in && !config->cluster_iam_master) { // Are we throttled and a slave?
-               master_throttle_packet(session[s].tbf_in, p, l); // Pass it to the master for handling.
+       // Are we throttled and a slave?
+       if (session[s].tbf_in && !config->cluster_iam_master) {
+               // Pass it to the master for handling.
+               master_throttle_packet(session[s].tbf_in, p, l);

                return;
        }
 
                return;
        }
 

-       if (session[s].tbf_in && config->cluster_iam_master) { // Are we throttled and a master?? actually handle the throttled packets.
+       // Are we throttled and a master??
+       if (session[s].tbf_in && config->cluster_iam_master) {
+               // Actually handle the throttled packets.

                tbf_queue_packet(session[s].tbf_in, p, l);
                return;
        }
                tbf_queue_packet(session[s].tbf_in, p, l);
                return;
        }


@@ -936,7 +945,7 @@ void processipv6in(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
 
        // Add on the tun header
        p -= 4;
 
        // Add on the tun header
        p -= 4;

-       *(uint32_t *)p = htonl(PKTIPV6);
+       *(uint32_t *) p = htonl(PKTIPV6);

        l += 4;
 
        // Are we throttled and a slave?
        l += 4;
 
        // Are we throttled and a slave?


@@ -946,9 +955,9 @@ void processipv6in(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
                return;
        }
 
                return;
        }
 

-       // Are we throttled and a master?? actually handle the throttled
-       // packets.
+       // Are we throttled and a master??

        if (session[s].tbf_in && config->cluster_iam_master) {
        if (session[s].tbf_in && config->cluster_iam_master) {

+               // Actually handle the throttled packets.

                tbf_queue_packet(session[s].tbf_in, p, l);
                return;
        }
                tbf_queue_packet(session[s].tbf_in, p, l);
                return;
        }


@@ -957,14 +966,16 @@ void processipv6in(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
        if (tun_write(p, l) < 0)
        {
                STAT(tun_tx_errors);
        if (tun_write(p, l) < 0)
        {
                STAT(tun_tx_errors);

-               LOG(0, s, t, "Error writing %d bytes to TUN device: %s" " (tunfd=%d, p=%p)\n",
+               LOG(0, s, t, "Error writing %d bytes to TUN device: %s (tunfd=%d, p=%p)\n",

                        l, strerror(errno), tunfd, p);
                        l, strerror(errno), tunfd, p);

+
+               return;

        }
 
        if (session[s].snoop_ip && session[s].snoop_port)
        {
                // Snooping this session
        }
 
        if (session[s].snoop_ip && session[s].snoop_port)
        {
                // Snooping this session

-               snoop_send_packet(p, l, session[s].snoop_ip, session[s].snoop_port);
+               snoop_send_packet(p + 4, l - 4, session[s].snoop_ip, session[s].snoop_port);

        }
 
        session[s].cin += l - 4;
        }
 
        session[s].cin += l - 4;


@@ -1094,7 +1105,7 @@ void sendchap(tunnelidt t, sessionidt s)
        radius[r].retry = backoff(radius[r].try++);
        if (radius[r].try > 5)
        {
        radius[r].retry = backoff(radius[r].try++);
        if (radius[r].try > 5)
        {

-               sessionshutdown(s, "Timeout CHAP");
+               sessionshutdown(s, "Timeout CHAP", 3, 0);

                STAT(tunnel_tx_errors);
                return ;
        }
                STAT(tunnel_tx_errors);
                return ;
        }