Make improvements in challenges.c

Co-authored-by: Ezequiel Bellver <ebellver@itba.edu.ar>
Co-authored-by: Juan Barmasch <jbarmasch@itba.edu.ar>

.gitignore

@@ -11,3 +11,4 @@ server
 client
 *.o
 quine
+a.out

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

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 = (rand() % (UPPER - LOWER + 1)) + LOWER;
-        int fdRandom = ((double) rand() / (RAND_MAX)) + 1;
         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
+            for (int i = (rand() % (6)) + 1; i > 0; i--) {
-            if (write(STDERR_FILENO, "=", 1) < 0)
+                int randNum = 90 * (rand() / (RAND_MAX + 1.0));
-                printSystemError("Challenges: write()");
+                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, "\33[0m\n", 5) < 0)
-    if (write(STDOUT_FILENO, "\033[1;1H\033[2J", 11) < 0)
         printSystemError("Challenges: write()");
 }
 
 void quineChallenge() {
-    while(1) {
+    int outputGCC = system("gcc quine.c -o quine");
-        char outputGCC = system("gcc quine.c -o quine");
+    if (outputGCC < 0)
-        if (!outputGCC) {
+        printSystemError("Challenges: system()");
-            while (1) {
+
-                printf("¡Genial!, ya lograron meter un programa en quine.c, veamos si hace lo que corresponde.\n");
+    if (outputGCC == 0) {
-                char outputDIFF = system("./quine | diff quine.c -");
+        printf("¡Genial!, ya lograron meter un programa en quine.c, veamos si hace lo que corresponde.\n");
-                if (!outputDIFF) {
+        int outputDIFF = system("./quine | diff quine.c -");
-                    printf("La respuesta es chin_chu_lan_cha\n");
+        if (outputDIFF < 0)
-                    return;
+            printSystemError("Challenges: system()");
-                }
+
-                else {
+        if (outputDIFF == 0)
-                    printf("diff encontró diferencias.\n");
+            printf("La respuesta es chin_chu_lan_cha\n");
-                    char ans;
+        else
-                    while ((ans = fgetc(streamGen)) != EOF && ans != '\n');
+            printf("\ndiff encontró diferencias.\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()");
-            }
-        }
     }
+    else
+        printf("\nENTER para reintentar.\n");
 }
 
 void gdbme(char * output) {
@@ -97,20 +82,11 @@ void gdbme(char * output) {
 }
 
 void gdbChallenge() {
-    while(1) {
+    char gdbOutput;
-        char gdbOutput;
+    gdbme(&gdbOutput);
-        gdbme(&gdbOutput);
+
-        if (gdbOutput) {
+    if (gdbOutput)
-            printf("La respuesta es gdb_rules\n");
+        printf("La respuesta es gdb_rules\n");
-            return;
+    else
-        }
+        printf("ENTER para reintentar.\n");
-        else {
-            printf("ENTER para reintentar.\n");
-            char ans;
-            while ((ans = fgetc(streamGen)) != EOF && ans != '\n');
-            if (ans == EOF) {
-                printSystemError("Challenge: fgetc()");
-            }
-        }
-    }
 }

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

include/server.h

@@ -9,8 +9,9 @@
 #include <unistd.h>
 #include <string.h>
 #include <stdlib.h>
-#include "errors.h"
+#include <time.h>
 #include <netinet/in.h>
+#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 \
+    {"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?",
+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},
+    {"quine", "¿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},
+    {"b gdbme y encontrá el valor mágico", "¿Qué es un RFC?", "gdb_rules\n", gdbChallenge},
 };
 
-char too_easy = 'a';
-
 void startChallenge();
 
 #endif

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++;
         }