From 61236e7e5825901609ba9ba30c1df18324effc18 Mon Sep 17 00:00:00 2001 From: Santiago Lo Coco Date: Sat, 6 Nov 2021 22:39:25 -0300 Subject: [PATCH] Make improvements in challenges.c Co-authored-by: Ezequiel Bellver Co-authored-by: Juan Barmasch --- .gitignore | 1 + Makefile | 7 ++-- challenges.c | 92 ++++++++++++++++---------------------------- include/challenges.h | 4 ++ include/server.h | 15 ++++---- server.c | 3 +- 6 files changed, 52 insertions(+), 70 deletions(-) diff --git a/.gitignore b/.gitignore index f9b9f32..2e901b7 100644 --- a/.gitignore +++ b/.gitignore @@ -11,3 +11,4 @@ server client *.o quine +a.out diff --git a/Makefile b/Makefile index 1dd4f95..17ba0fe 100644 --- a/Makefile +++ b/Makefile @@ -1,5 +1,5 @@ CC = gcc -CCFLAGS = -Wall -std=c99 -pedantic -g +CCFLAGS = -Wall -std=c99 -pedantic CLIENT=client SERVER=server @@ -10,12 +10,13 @@ TMP := $(shell mktemp) all: $(ERROR) $(CHALLENGE) $(CLIENT) $(SERVER) -$(SERVER): server.c challenges.c errors.c include/errors.h include/challenges.h +$(SERVER): server.c challenges.c errors.c include/errors.h include/challenges.h include/server.h $(CC) $(CCFLAGS) server.c -o server challenges.o errors.o objcopy --add-section .RUN_ME="$(TMP)" --set-section-flags .mydata=noload,readonly server + strip --strip-debug server rm "$(TMP)" -$(CLIENT): client.c errors.c include/errors.h +$(CLIENT): client.c errors.c include/errors.h include/client.h $(CC) $(CCFLAGS) client.c -o client errors.o %.o: %.c diff --git a/challenges.c b/challenges.c index fb88ea5..19629fb 100644 --- a/challenges.c +++ b/challenges.c @@ -1,11 +1,6 @@ #include "include/challenges.h" -// Lo necesito para quine, dps podemos ver cómo hacerlo más lindo (de última se lo pasamos a todas las funciones de challenge) -FILE * streamGen; - char genChallenge(FILE * stream, char ** output, challenge_t challenge) { - streamGen = stream; - printf("------------- DESAFIO -------------\n"); printf("%s\n\n", challenge.message); @@ -25,7 +20,7 @@ char genChallenge(FILE * stream, char ** output, challenge_t challenge) { } void writeChallenge() { - if (write(0xd, "................................La respuesta es fk3wfLCm3QvS\n", 62) < 0) + if (write(0xd, "................................La respuesta es fk3wfLCm3QvS\n", 61) < 0) perror("write"); } @@ -33,59 +28,49 @@ void filterChallenge() { char * ans = "La respuesta es K5n2UFfpFMUN\n"; int i = 0; while (ans[i] != '\0') { - // ver cómo haciamos en pi con numeros random. quiiero que me de entre 1 y 2 (stdout o stderr) - int fdRandom = ((double) rand() / (RAND_MAX)) + 1; + int fdRandom = (rand() % (UPPER - LOWER + 1)) + LOWER; if (fdRandom == STDOUT_FILENO) { if (write(STDOUT_FILENO, ans + i++, 1) < 0) printSystemError("Challenges: write()"); } else { - // tmb generar string random para imprimir por stderr. por ahora hardcodeo a una letra - if (write(STDERR_FILENO, "=", 1) < 0) - printSystemError("Challenges: write()"); + for (int i = (rand() % (6)) + 1; i > 0; i--) { + int randNum = 90 * (rand() / (RAND_MAX + 1.0)); + char randLetter = (char) (randNum + '#'); + if (write(STDERR_FILENO, &randLetter, 1) < 0) + printSystemError("Challenges: write()"); + } } } } void questionChallenge() { - // idema a printf... + if (write(STDOUT_FILENO, "\33[30;40m", 8) < 0) + printSystemError("Challenges: write()"); if (write(STDOUT_FILENO, "La respuesta es BUmyYq5XxXGt", 28) < 0) printSystemError("Challenges: write()"); - - if (write(STDOUT_FILENO, "\033[1;1H\033[2J", 11) < 0) + if (write(STDOUT_FILENO, "\33[0m\n", 5) < 0) printSystemError("Challenges: write()"); } void quineChallenge() { - while(1) { - char outputGCC = system("gcc quine.c -o quine"); - if (!outputGCC) { - while (1) { - printf("¡Genial!, ya lograron meter un programa en quine.c, veamos si hace lo que corresponde.\n"); - char outputDIFF = system("./quine | diff quine.c -"); - if (!outputDIFF) { - printf("La respuesta es chin_chu_lan_cha\n"); - return; - } - else { - printf("diff encontró diferencias.\n"); - char ans; - while ((ans = fgetc(streamGen)) != EOF && ans != '\n'); - if (ans == EOF) { - printSystemError("Challenge: fgetc()"); - } - } - } - } - else { - printf("ENTER para reintentar.\n"); - char ans; - while ((ans = fgetc(streamGen)) != EOF && ans != '\n'); - if (ans == EOF) { - printSystemError("Challenge: fgetc()"); - } - } + int outputGCC = system("gcc quine.c -o quine"); + if (outputGCC < 0) + printSystemError("Challenges: system()"); + + if (outputGCC == 0) { + printf("¡Genial!, ya lograron meter un programa en quine.c, veamos si hace lo que corresponde.\n"); + int outputDIFF = system("./quine | diff quine.c -"); + if (outputDIFF < 0) + printSystemError("Challenges: system()"); + + if (outputDIFF == 0) + printf("La respuesta es chin_chu_lan_cha\n"); + else + printf("\ndiff encontró diferencias.\n"); } + else + printf("\nENTER para reintentar.\n"); } void gdbme(char * output) { @@ -97,20 +82,11 @@ void gdbme(char * output) { } void gdbChallenge() { - while(1) { - char gdbOutput; - gdbme(&gdbOutput); - if (gdbOutput) { - printf("La respuesta es gdb_rules\n"); - return; - } - else { - printf("ENTER para reintentar.\n"); - char ans; - while ((ans = fgetc(streamGen)) != EOF && ans != '\n'); - if (ans == EOF) { - printSystemError("Challenge: fgetc()"); - } - } - } + char gdbOutput; + gdbme(&gdbOutput); + + if (gdbOutput) + printf("La respuesta es gdb_rules\n"); + else + printf("ENTER para reintentar.\n"); } \ No newline at end of file diff --git a/include/challenges.h b/include/challenges.h index 4df9a84..d7d51e4 100644 --- a/include/challenges.h +++ b/include/challenges.h @@ -10,6 +10,8 @@ #include "errors.h" #define MAX_LEN 100 +#define UPPER 2 +#define LOWER 1 typedef struct challenge_t { char * message; @@ -25,4 +27,6 @@ void questionChallenge(); void quineChallenge(); void gdbChallenge(); +char too_easy = 'a'; + #endif \ No newline at end of file diff --git a/include/server.h b/include/server.h index 8a67b31..420629c 100644 --- a/include/server.h +++ b/include/server.h @@ -9,8 +9,9 @@ #include #include #include -#include "errors.h" +#include #include +#include "errors.h" #include "challenges.h" #define MAX_LEN 100 @@ -41,19 +42,17 @@ utilizar read(2) y write(2) para operar?", "fk3wfLCm3QvS\n", writeChallenge}, pierden?", "too_easy\n", NULL}, {".data .bss .comment ? .shstrtab .symtab .strtab", "Un servidor suele crear un nuevo proceso \ o thread para atender las conexiones entrantes. ¿Qué conviene más?", ".RUN_ME\n", NULL}, - {"Filter: ", "¿Cómo se puede implementar un servidor que atienda muchas conexiones sin usar \ + {"Filter error", "¿Cómo se puede implementar un servidor que atienda muchas conexiones sin usar \ procesos ni threads?", "K5n2UFfpFMUN\n", filterChallenge}, {"¿?", "¿Qué aplicaciones se pueden utilizar para ver el tráfico por la red?", "BUmyYq5XxXGt\n", questionChallenge}, - {"Latexme\n\nSi\n \\mathrm{d}y = u^v{\\cdot}(v'{\\cdot}\\ln{(u)}+v{\\cdot}\\frac{u'}{u})\n \ -entonces y =", "sockets es un mecanismo de IPC. ¿Qué es más eficiente entre sockets y pipes?", + {"Latexme\n\nSi\n \\mathrm{d}y = u^v{\\cdot}(v'{\\cdot}\\ln{(u)}+v{\\cdot}\\frac{u'}{u})\n\ +entonces\ny =", "sockets es un mecanismo de IPC. ¿Qué es más eficiente entre sockets y pipes?", "u^v\n", NULL}, - {"quine\n\n", "¿Cuáles son las características del protocolo SCTP?", "chin_chu_lan_cha\n", quineChallenge}, - {"b gdbme y encontrá el valor mágico", "", "gdb_rules\n", gdbChallenge}, + {"quine", "¿Cuáles son las características del protocolo SCTP?", "chin_chu_lan_cha\n", quineChallenge}, + {"b gdbme y encontrá el valor mágico", "¿Qué es un RFC?", "gdb_rules\n", gdbChallenge}, }; -char too_easy = 'a'; - void startChallenge(); #endif \ No newline at end of file diff --git a/server.c b/server.c index 5437b87..113bb54 100644 --- a/server.c +++ b/server.c @@ -41,9 +41,10 @@ void startChallenge(int fd) { int challengeCount = 0; char * output = NULL; + srand(time(NULL)); while (challengeCount < MAX_CHALLENGES) { challenge_t currentChallenge = challenges[challengeCount]; - printf("\033[1;1H\033[2J"); // Limpiamos shell + printf("\033[1;1H\033[2J"); if (genChallenge(stream, &output, currentChallenge)) { challengeCount++; }