compte rendutp2.txt

@@ -1,31 +0,0 @@
-Compte rendu 
-
-daemon.c : 
-
-create_daemon() :
-La complexité est de 4 car il y a les 2 boucles if et la boucle for et +1 s'il se passe rien
-
-ping_request() : La complexite est de 2 ca il y'a une boucle if + 1 s'il ne se passe rien
-
-send_check() : La complexite est de 3 car il y'a 2 boucles if + 1 s'il se passe rien.
-
-check_keep_working() : La complexite est de 4 car il y a le premier if, le 2eme if et dans le 3eme if on compte les 2 else.
-
-daemon_work() : Le copmlexité est de 3 si if(db_connect()) fonctionne et fonctionne pas et +1 s'il ne se passe rien.
-
-
-db-sqlite.c :
-
-db_connect() : La complexité est de 1 car il y'a juste le return.
-db_disconnect() : La complexité est de 1 également juste le return.
-insert_hourly_report : La complexité est de 1 car il n'y a aucune boucle
-
-
-ping-report.c :
-
-La complexité est de 5 car il y a 5 chemins possible dans celui-ci.
-
-
-stats.c :
-
-get_ping-from_temp_log : La complexité est de 13 car 13 chemins. (Voir l'arbre)

ping-report/src/stats.c

@@ -22,70 +22,104 @@
     Return value :
         Ping value as a string or NULL if an error occured
 */
-/*@null@*/char* get_ping_from_temp_log() {
+/*@null@*/char* get_ping_from_temp_log(){
+
+    /* Variables */
     FILE* fd = NULL;
     char* read_line = NULL;
     size_t n = 0;
     size_t nmatch = 2;
-    regex_t* p_reg = malloc(sizeof(*p_reg));
-    regmatch_t* pmatch = NULL;
+    regex_t *p_reg;
+    regmatch_t* pmatch;
     char* ping = NULL;
+    int start;
+    int end;
+    size_t size_ping;
 
-    if (!p_reg) return NULL;
+    /* regex struct memory allocation */
+    p_reg = (regex_t *) malloc(sizeof(*p_reg));
+    if(p_reg == NULL){
+        return ping; /* NULL */
+    }
 
-    fd = fopen("/var/log/ping-report/last-ping.log", "r");
-    if (!fd) {
+    /* Open ping log file */
+    fd = fopen("/var/log/ping-report/last-ping.log","r");
+    if(fd == NULL){
         free(p_reg);
-        return NULL;
+        return ping; /* NULL */
     }
 
-    if (regcomp(p_reg, "time=(.*) ms", REG_EXTENDED) != 0) {
-        cleanup(fd, p_reg, NULL, NULL);
-        return NULL;
-    }
-
-    pmatch = malloc(sizeof(*pmatch) * nmatch);
-    if (!pmatch) {
-        cleanup(fd, p_reg, NULL, NULL);
-        return NULL;
-    }
-
-    while (getline(&read_line, &n, fd) != -1) {
-        if (regexec(p_reg, read_line, nmatch, pmatch, 0) == 0) {
-            ping = extract_ping(read_line, pmatch);
-            free(read_line);
-            break;
+    /* Construct regex to get ping from log file */
+    if(regcomp(p_reg,"time=(.*) ms",REG_EXTENDED) != 0){
+        if(p_reg != NULL){
+            free(p_reg);
         }
-        free(read_line);
-        read_line = NULL;
+        (void) fclose(fd);
+        return ping; /* NULL */
     }
 
-    cleanup(fd, p_reg, pmatch, read_line);
-    return ping;
-}
-
-void cleanup(FILE* fd, regex_t* p_reg, regmatch_t* pmatch, char* read_line) {
-    if (fd) fclose(fd);
-    if (p_reg) {
+    /* match info memory allocation */
+    pmatch = malloc(sizeof(*pmatch) * nmatch);
+    if(pmatch == NULL){
+        (void) fclose(fd);
         regfree(p_reg);
         free(p_reg);
-    }
-    if (pmatch) free(pmatch);
-    if (read_line) free(read_line);
-}
-
-char* extract_ping(char* read_line, regmatch_t* pmatch) {
-    int start = (int)pmatch[1].rm_so;
-    int end = (int)pmatch[1].rm_eo;
-    size_t size_ping = (size_t)(end - start);
-    char* ping = malloc(sizeof(char) * (size_ping + 2));
-
-    if (ping) {
-        strncpy(ping, &read_line[start], size_ping);
-        ping[size_ping] = '\n';
-        ping[size_ping + 1] = '\0';
+        return ping; /* NULL */
     }
 
+    /* Read file */
+    while(getline(&read_line,&n,fd) != -1){
+        
+        if(read_line == NULL){
+            break;
+        }
+
+        /* Exec regex to find ping */
+
+        if(regexec(p_reg,read_line,nmatch,pmatch,0) == 0){
+
+            /* Extract ping position from read line */
+            start = (int) pmatch[1].rm_so;
+            end = (int) pmatch[1].rm_eo;
+            size_ping = (size_t) (end - start);
+
+            /* ping string memory allocation */
+            ping = malloc(sizeof(char) * (size_ping+2));
+            if(ping == NULL){
+                free(read_line);
+                read_line = NULL;
+                n = 0;
+                break;
+            }
+
+            /* Create ping string */
+            (void) strncpy(ping, &read_line[start], size_ping);
+            ping[size_ping]='\n';
+            ping[size_ping+1]='\0';
+
+            /* Free memory */
+            free(read_line);
+            read_line = NULL;
+            n = 0;
+            break;
+        }
+
+        free(read_line);
+        read_line = NULL;
+        n = 0;
+    }
+
+    /* free allocated memory */
+    regfree(p_reg);
+    free(p_reg);
+    free(pmatch);
+    if(read_line != NULL){
+        free(read_line);
+    }
+
+    (void) fclose(fd);
+
+    /* ping may be null, then it must mean that the ping request was lost */
     return ping;
 }