diff --git a/README.md b/README.md
index 53b2246..9a854fc 100644
--- a/README.md
+++ b/README.md
@@ -27,7 +27,7 @@ make all
Ahora, tendrá dos ejecutables: `client` y `socks5d`.
-El servidor por defecto utilizará un _TOKEN_ que debe ser una `env` variable. Por ejemplo, puede configurar una mediante:
+El servidor por defecto utilizará un _TOKEN_ que debe ser una `environment` variable. Por ejemplo, puede configurar una mediante:
```bash
export BPROXY_TOKEN=0x0FF1CEDEADB00B1E
@@ -47,11 +47,13 @@ y en otra terminal
./client ${PARAMS}
```
+Debe notar que el server espera un token como el que se mencionó anteriormente (en formato hexadecimal) mientras que el cliente espera un número decimal (véase la siguiente sección para más información).
+
## Casos de uso
Se mostrarán dos casos típicos de uso del servidor.
-Aquí se tiene al servidor proxy corriendo en 127.0.0.1:1080 y se utiliza a este para conectarse a 192.168.10.120:110 (una PC en la misma subred corriendo un servidor POP3 local). Además, se está autenticando con el usuario `user:pass`.
+Aquí se tiene al servidor proxy corriendo en `127.0.0.1:1080` y se utiliza a este para conectarse a `192.168.10.120:110` (una PC corriendo un servidor POP3). Además, se está autenticando con el usuario `user:pass`.
```bash
./socks5d -u user:pass
@@ -65,14 +67,24 @@ Por otro lado, aquí se usó `curl` para acceder a http://api.ipify.org mediante
curl -x socks5://user:pass@127.0.0.1:1080 http://api.ipify.org
```
-En este caso se quiere obtener la cantidad de páginas de usuarios que hay en el servidor proxy, mediante -L se especifica la direccion IP del servidor de configuración (por defecto 127.0.0.1), con -P se especifica el puerto (por defecto 8080).
-Una vez que se obtiene la cantidad de páginas se le pide el listado de usuarios pertenecientes a la 2da página, en caso de que el servidor no cuente con usuarios suficientes para tener 2 páginas, imprimirá un error especificando que el valor pasado no es válido. En cambio, si se cuentan con usuarios suficientes se listarán separados por un '\n'
+Ahora, se mostrará un caso de uso para el cliente de configuración.
+
+Se quiere obtener la cantidad de páginas de usuarios que hay en el servidor proxy, mediante -L se especifica la direccion IP del servidor de configuración (por defecto `127.0.0.1`), con -P se especifica el puerto (por defecto `8080`).
```bash
+export BPROXY_TOKEN=0x0FF1CEDEADB00B1E
+./socks5d -u user:pass
./client 1148926835748244254 -L 192.168.10.119 -P 8100 -f
+```
+
+Una vez que se obtiene la cantidad de páginas se le pide el listado de usuarios pertenecientes a la 2da página, en caso de que el servidor no cuente con usuarios suficientes para tener 2 páginas, imprimirá un error especificando que el valor pasado no es válido. En cambio, si se cuentan con usuarios suficientes se listarán separados por un '\n'.
+
+```bash
./client 1148926835748244254 -L 192.168.10.119 -P 8100 -u 2
```
+Note que `0x0FF1CEDEADB00B1E` (base 16) = `1148926835748244254` (base 10).
+
## Testeos
En orden de realizar un análisis estático del sistema usted debe tener instalado [cppcheck](http://cppcheck.net/) y [pvs-studio](https://pvs-studio.com/). Luego, puede correrlos con:
@@ -87,6 +99,8 @@ Por último, si quiere hacer un análisis dinámico (usando [valgrind](https://v
valgrind ./socks5d
```
+Debe tener el cuidado de recompilarlo eliminando los flags de compilación `-fsanitize=address -fno-omit-frame-pointer`.
+
## Limpieza
Si desea borrar los archivos creados luego de la compilación (y los testeos si se hubiesen corrido) debe correr:
@@ -95,7 +109,7 @@ make clean
```
## Documentación
-Debe notar que el informe y el rfc del protocolo BCP se encuentran en la carpeta `docs/`.
+Debe notar que el informe y el rfc del protocolo BCP se encuentran en la carpeta `docs/` (junto con el `man` de socks5d).
# Autores
### Bottler:
diff --git a/docs/informe.pdf b/docs/informe.pdf
new file mode 100644
index 0000000..9ba550a
Binary files /dev/null and b/docs/informe.pdf differ
diff --git a/src/args.c b/src/args.c
index 1a61135..d662b11 100644
--- a/src/args.c
+++ b/src/args.c
@@ -46,7 +46,7 @@ static void user(char * s, user_t * user) {
}
static void version(void) {
- fprintf(stderr, "BProxy v1.0\nITBA Protocolos de Comunicación 2021/1 -- Group 7\nSee LICENSE.md\n");
+ fprintf(stderr, "BProxy v1.0\nITBA Protocolos de Comunicación 2022/1C -- Group 7\nSee LICENSE.md\n");
}
static void usage(const char *progname) {
@@ -60,7 +60,7 @@ static void usage(const char *progname) {
" -u : Username and password of proxy user. Up to 1024.\n"
" -v Prints version information.\n\n",
progname);
- exit(1);
+ exit(EXIT_SUCCESS);
}
static void print_logo() {
@@ -81,8 +81,6 @@ static void print_logo() {
}
void parse_args(const int argc, char ** argv, struct socks5args * args) {
- print_logo();
-
memset(args, 0, sizeof(*args));
args->socks_addr = NULL;
@@ -127,7 +125,7 @@ void parse_args(const int argc, char ** argv, struct socks5args * args) {
break;
case 'v':
version();
- exit(0);
+ exit(EXIT_SUCCESS);
default:
fprintf(stderr, "Unknown argument %d.\n", c);
exit(1);
@@ -144,4 +142,6 @@ void parse_args(const int argc, char ** argv, struct socks5args * args) {
fprintf(stderr, "\n");
exit(1);
}
+
+ print_logo();
}
diff --git a/src/client.c b/src/client.c
index 57bb211..a1ce841 100644
--- a/src/client.c
+++ b/src/client.c
@@ -39,7 +39,7 @@ static unsigned short get_port(const char *s) {
}
static void version(void) {
- fprintf(stdout, "BProxy v1.0\nITBA Protocolos de Comunicación 2021/1 -- Group 7\nSee LICENSE.md\n");
+ fprintf(stdout, "BCP Client v1.0\nITBA Protocolos de Comunicación 2022/1C -- Group 7\nSee LICENSE.md\n");
}
static void usage(const char * progname) {
@@ -59,10 +59,10 @@ static void usage(const char * progname) {
" [TOKEN] -a Get proxy authentication status\n"
" [TOKEN] -L Address of management service.\n"
" [TOKEN] -P Port of management service.\n"
- " [TOKEN] -A Modify proxy authentication status\n"
- "Where TOKEN is an 8-byte unsigned integer\n\n",
+ " [TOKEN] -A Modify proxy authentication status\n\n"
+ "Where TOKEN is an 8-byte unsigned decimal integer.\n\n",
progname);
- exit(1);
+ exit(EXIT_SUCCESS);
}
uint8_t parse_args(int argc, char ** argv, uint8_t * buffer, char ** address, unsigned short * port) {
diff --git a/src/server.c b/src/server.c
index 71761c6..4d3ceee 100644
--- a/src/server.c
+++ b/src/server.c
@@ -35,19 +35,9 @@ static struct socks5args * args;
uint64_t config_token;
int main(int argc, char ** argv) {
- char * token = getenv("BPROXY_TOKEN");
- if (token != NULL) {
- config_token = strtoul(token, NULL, 16);
- }
- else {
- fprintf(stderr, "No token found. Please see README.md\n");
- exit(EXIT_FAILURE);
- }
args = malloc(sizeof(struct socks5args));
parse_args(argc, argv, args);
- close(STDIN_FILENO);
-
const char *err_msg = NULL;
selector_status ss = SELECTOR_SUCCESS;
fd_selector selector = NULL;
@@ -58,6 +48,17 @@ int main(int argc, char ** argv) {
int udp_server = -1;
int udp_server_ipv6 = -1;
+ char * token = getenv("BPROXY_TOKEN");
+ if (token != NULL) {
+ config_token = strtoul(token, NULL, 16);
+ }
+ else {
+ err_msg = "No token found. Please see README.md";
+ goto finally;
+ }
+
+ close(STDIN_FILENO);
+
unsigned char buf[sizeof(struct in6_addr)];
int domain = -1;
if (args->socks_addr != NULL) {