allow username to be set when ungardening

This commit is contained in:
bodea 2005-03-10 03:31:25 +00:00
parent 466ffcdaac
commit 29c173cadc
4 changed files with 28 additions and 15 deletions

View file

@ -33,6 +33,8 @@
- Avoid a possible buffer overflow in processpap. - Avoid a possible buffer overflow in processpap.
- Kill session if authentication was rejected. - Kill session if authentication was rejected.
- Simplify AVP unhiding code. - Simplify AVP unhiding code.
- Add optional "username" parameter to ungarden control, allowing the
username to be reset before going online.
* Fri Dec 17 2004 Brendan O'Dea <bod@optusnet.com.au> 2.0.13 * Fri Dec 17 2004 Brendan O'Dea <bod@optusnet.com.au> 2.0.13
- Better cluster master collision resolution: keep a counter of state - Better cluster master collision resolution: keep a counter of state

View file

@ -9,7 +9,7 @@
/* walled garden */ /* walled garden */
char const *cvs_id = "$Id: garden.c,v 1.20 2004/12/16 08:49:53 bodea Exp $"; char const *cvs_id = "$Id: garden.c,v 1.21 2005/03/10 03:31:25 bodea Exp $";
int plugin_api_version = PLUGIN_API_VERSION; int plugin_api_version = PLUGIN_API_VERSION;
static struct pluginfuncs *p = 0; static struct pluginfuncs *p = 0;
@ -44,7 +44,7 @@ char *down_commands[] = {
#define F_GARDEN 1 #define F_GARDEN 1
#define F_CLEANUP 2 #define F_CLEANUP 2
int garden_session(sessiont *s, int flag); int garden_session(sessiont *s, int flag, char *newuser);
int plugin_post_auth(struct param_post_auth *data) int plugin_post_auth(struct param_post_auth *data)
{ {
@ -63,7 +63,7 @@ int plugin_new_session(struct param_new_session *data)
return PLUGIN_RET_OK; // Slaves don't do walled garden processing. return PLUGIN_RET_OK; // Slaves don't do walled garden processing.
if (data->s->walled_garden) if (data->s->walled_garden)
garden_session(data->s, F_GARDEN); garden_session(data->s, F_GARDEN, 0);
return PLUGIN_RET_OK; return PLUGIN_RET_OK;
} }
@ -74,14 +74,14 @@ int plugin_kill_session(struct param_new_session *data)
return PLUGIN_RET_OK; // Slaves don't do walled garden processing. return PLUGIN_RET_OK; // Slaves don't do walled garden processing.
if (data->s->walled_garden) if (data->s->walled_garden)
garden_session(data->s, F_CLEANUP); garden_session(data->s, F_CLEANUP, 0);
return PLUGIN_RET_OK; return PLUGIN_RET_OK;
} }
char *plugin_control_help[] = { char *plugin_control_help[] = {
" garden USER|SID Put user into the walled garden", " garden USER|SID Put user into the walled garden",
" ungarden SID Release session from garden", " ungarden SID [USER] Release session from garden",
0 0
}; };
@ -103,10 +103,13 @@ int plugin_control(struct param_control *data)
flag = data->argv[0][0] == 'g' ? F_GARDEN : F_UNGARDEN; flag = data->argv[0][0] == 'g' ? F_GARDEN : F_UNGARDEN;
if (data->argc != 2) if (data->argc < 2 || data->argc > 3 || (data->argc > 2 && flag == F_GARDEN))
{ {
data->response = NSCTL_RES_ERR; data->response = NSCTL_RES_ERR;
data->additional = "one argument required: username or session id"; data->additional = flag == F_GARDEN
? "requires username or session id"
: "requires session id and optional username";
return PLUGIN_RET_STOP; return PLUGIN_RET_STOP;
} }
@ -135,7 +138,7 @@ int plugin_control(struct param_control *data)
return PLUGIN_RET_STOP; return PLUGIN_RET_STOP;
} }
garden_session(s, flag); garden_session(s, flag, data->argc > 2 ? data->argv[2] : 0);
p->session_changed(session); p->session_changed(session);
data->response = NSCTL_RES_OK; data->response = NSCTL_RES_OK;
@ -162,12 +165,12 @@ int plugin_become_master(void)
int plugin_new_session_master(sessiont *s) int plugin_new_session_master(sessiont *s)
{ {
if (s->walled_garden) if (s->walled_garden)
garden_session(s, F_GARDEN); garden_session(s, F_GARDEN, 0);
return PLUGIN_RET_OK; return PLUGIN_RET_OK;
} }
int garden_session(sessiont *s, int flag) int garden_session(sessiont *s, int flag, char *newuser)
{ {
char cmd[2048]; char cmd[2048];
sessionidt sess; sessionidt sess;
@ -191,6 +194,12 @@ int garden_session(sessiont *s, int flag)
// Normal User // Normal User
p->log(2, sess, s->tunnel, "Un-Garden user %s (%s)\n", s->user, p->fmtaddr(htonl(s->ip), 0)); p->log(2, sess, s->tunnel, "Un-Garden user %s (%s)\n", s->user, p->fmtaddr(htonl(s->ip), 0));
if (newuser)
{
snprintf(s->user, MAXUSER, "%s", newuser);
p->log(2, sess, s->tunnel, " Setting username to %s\n", s->user);
}
// Kick off any duplicate usernames // Kick off any duplicate usernames
// but make sure not to kick off ourself // but make sure not to kick off ourself
if (s->ip && !s->die && (other = p->get_session_by_username(s->user)) && s != p->get_session_by_id(other)) { if (s->ip && !s->die && (other = p->get_session_by_username(s->user)) && s != p->get_session_by_id(other)) {

View file

@ -1,5 +1,5 @@
// L2TPNS Global Stuff // L2TPNS Global Stuff
// $Id: l2tpns.h,v 1.57 2005/02/14 06:58:39 bodea Exp $ // $Id: l2tpns.h,v 1.58 2005/03/10 03:31:25 bodea Exp $
#ifndef __L2TPNS_H__ #ifndef __L2TPNS_H__
#define __L2TPNS_H__ #define __L2TPNS_H__
@ -32,6 +32,8 @@
#define MAXCONTROL 1000 // max length control message we ever send... #define MAXCONTROL 1000 // max length control message we ever send...
#define MAXETHER (1500+18) // max packet we try sending to tun #define MAXETHER (1500+18) // max packet we try sending to tun
#define MAXTEL 96 // telephone number #define MAXTEL 96 // telephone number
#define MAXUSER 128 // username
#define MAXPASS 128 // password
#define MAXPLUGINS 20 // maximum number of plugins to load #define MAXPLUGINS 20 // maximum number of plugins to load
#define MAXRADSERVER 10 // max radius servers #define MAXRADSERVER 10 // max radius servers
#define MAXROUTE 10 // max static routes per session #define MAXROUTE 10 // max static routes per session
@ -202,7 +204,7 @@ typedef struct
uint8_t flags1; // additional flags (currently unused); uint8_t flags1; // additional flags (currently unused);
char random_vector[MAXTEL]; char random_vector[MAXTEL];
int random_vector_length; int random_vector_length;
char user[129]; // user (needed in seesion for radius stop messages) (can we reduce this? --mo) char user[MAXUSER]; // user (needed in seesion for radius stop messages)
char called[MAXTEL]; // called number char called[MAXTEL]; // called number
char calling[MAXTEL]; // calling number char calling[MAXTEL]; // calling number
uint32_t tx_connect_speed; uint32_t tx_connect_speed;

6
ppp.c
View file

@ -1,6 +1,6 @@
// L2TPNS PPP Stuff // L2TPNS PPP Stuff
char const *cvs_id_ppp = "$Id: ppp.c,v 1.44 2005/02/14 06:58:39 bodea Exp $"; char const *cvs_id_ppp = "$Id: ppp.c,v 1.45 2005/03/10 03:31:25 bodea Exp $";
#include <stdio.h> #include <stdio.h>
#include <string.h> #include <string.h>
@ -28,8 +28,8 @@ static void initccp(tunnelidt t, sessionidt s);
// Process PAP messages // Process PAP messages
void processpap(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l) void processpap(tunnelidt t, sessionidt s, uint8_t *p, uint16_t l)
{ {
char user[129]; char user[MAXUSER];
char pass[129]; char pass[MAXPASS];
uint16_t hl; uint16_t hl;
CSTAT(processpap); CSTAT(processpap);