From ae4eb5db87753dae7a40264650bd15984f53e93e Mon Sep 17 00:00:00 2001
From: Santiago Lo Coco <slococo@itba.edu.ar>
Date: Tue, 2 Nov 2021 23:39:01 -0300
Subject: [PATCH] Add PVS-Studio and cppcheck (and make some fixes detailed in
 Informe.docx)

Co-authored-by: Ezequiel Bellver <ebellver@itba.edu.ar>
Co-authored-by: Juan Barmasch <jbarmasch@itba.edu.ar>
---
 Bootloader/BMFS/bmfs.c                        |   2 +
 Kernel/drivers/include/naiveConsole.h         |  24 ++--
 Kernel/drivers/keyboard.c                     |   5 +-
 Kernel/drivers/naiveConsole.c                 | 120 +++++++++---------
 Kernel/drivers/time.c                         |   2 +
 Kernel/drivers/video.c                        |   2 +
 Kernel/interruptions/exceptions.c             |   2 +
 Kernel/interruptions/idtLoader.c              |   2 +
 Kernel/interruptions/irqDispatcher.c          |   2 +
 Kernel/interruptions/systemCalls.c            |   8 +-
 Kernel/interruptions/systemCallsDispatcher.c  |  12 +-
 Kernel/kernel.c                               |   2 +
 Kernel/lib.c                                  |   2 +
 Kernel/moduleLoader.c                         |   2 +
 Kernel/utils/include/pipeLib.h                |   1 +
 Kernel/utils/include/schedulerLib.h           |   2 +
 Kernel/utils/memManagerBuddy.c                |  43 +------
 Kernel/utils/memManagerFRT4.c                 |   3 +-
 Kernel/utils/pipe.c                           |   2 +
 Kernel/utils/scheduler.c                      |  13 +-
 Kernel/utils/sem.c                            |   4 +
 Makefile                                      |   8 +-
 Toolchain/ModulePacker/main.c                 |   2 +
 Userland/SampleCodeModule/_loader.c           |   2 +
 Userland/SampleCodeModule/asm/libasm.asm      |  20 ++-
 Userland/SampleCodeModule/bottler/bottler.c   |   2 +
 Userland/SampleCodeModule/include/system.h    |   2 +
 Userland/SampleCodeModule/libc.c              |   4 +-
 Userland/SampleCodeModule/sampleCodeModule.c  |   2 +
 .../SampleCodeModule/shell/commands/block.c   |   2 +
 .../SampleCodeModule/shell/commands/cat.c     |  12 +-
 .../SampleCodeModule/shell/commands/clear.c   |   2 +
 .../SampleCodeModule/shell/commands/cpuid.c   |   2 +
 .../SampleCodeModule/shell/commands/excDiv.c  |   2 +
 .../SampleCodeModule/shell/commands/excOP.c   |   2 +
 .../SampleCodeModule/shell/commands/filter.c  |   4 +
 .../SampleCodeModule/shell/commands/help.c    |   2 +
 .../SampleCodeModule/shell/commands/inforeg.c |   2 +
 .../SampleCodeModule/shell/commands/kill.c    |   2 +
 .../SampleCodeModule/shell/commands/loop.c    |   2 +
 .../SampleCodeModule/shell/commands/mem.c     |   2 +
 .../SampleCodeModule/shell/commands/nice.c    |   2 +
 .../SampleCodeModule/shell/commands/phylo.c   |   3 +-
 .../SampleCodeModule/shell/commands/pipe.c    |   2 +
 .../shell/commands/printmem.c                 |   2 +
 Userland/SampleCodeModule/shell/commands/ps.c |   2 +
 .../shell/commands/quadratic.c                |   2 +
 .../SampleCodeModule/shell/commands/sems.c    |   2 +
 .../SampleCodeModule/shell/commands/test_mm.c |   4 +
 .../shell/commands/test_prio.c                |   2 +
 .../shell/commands/test_processes.c           |   6 +-
 .../shell/commands/test_sync.c                |   9 +-
 .../shell/commands/test_util.c                |   2 +
 .../SampleCodeModule/shell/commands/time.c    |   2 +
 .../SampleCodeModule/shell/commands/unblock.c |   2 +
 Userland/SampleCodeModule/shell/commands/wc.c |   3 +
 .../SampleCodeModule/shell/include/phyloLib.h |   3 +-
 Userland/SampleCodeModule/shell/shell.c       |  10 +-
 build.sh                                      |   6 +-
 flags.txt                                     |   2 +-
 60 files changed, 252 insertions(+), 147 deletions(-)

diff --git a/Bootloader/BMFS/bmfs.c b/Bootloader/BMFS/bmfs.c
index eb4162b..6a69c95 100644
--- a/Bootloader/BMFS/bmfs.c
+++ b/Bootloader/BMFS/bmfs.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 /* BareMetal File System Utility */
 /* Written by Ian Seyler of Return Infinity */
 
diff --git a/Kernel/drivers/include/naiveConsole.h b/Kernel/drivers/include/naiveConsole.h
index 692335a..28adf52 100644
--- a/Kernel/drivers/include/naiveConsole.h
+++ b/Kernel/drivers/include/naiveConsole.h
@@ -1,15 +1,15 @@
-#ifndef NAIVE_CONSOLE_H
-#define NAIVE_CONSOLE_H
+// #ifndef NAIVE_CONSOLE_H
+// #define NAIVE_CONSOLE_H
 
-#include <stdint.h>
+// #include <stdint.h>
 
-void ncPrint(const char * string);
-void ncPrintChar(char character);
-void ncNewline();
-void ncPrintDec(uint64_t value);
-void ncPrintHex(uint64_t value);
-void ncPrintBin(uint64_t value);
-void ncPrintBase(uint64_t value, uint32_t base);
-void ncClear();
+// void ncPrint(const char * string);
+// void ncPrintChar(char character);
+// void ncNewline();
+// void ncPrintDec(uint64_t value);
+// void ncPrintHex(uint64_t value);
+// void ncPrintBin(uint64_t value);
+// void ncPrintBase(uint64_t value, uint32_t base);
+// void ncClear();
 
-#endif
\ No newline at end of file
+// #endif
\ No newline at end of file
diff --git a/Kernel/drivers/keyboard.c b/Kernel/drivers/keyboard.c
index bbc3226..c80c592 100644
--- a/Kernel/drivers/keyboard.c
+++ b/Kernel/drivers/keyboard.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "keyboard.h"
 
 #define SIZE 1
@@ -37,7 +39,8 @@ void testKeyboardInterrupt(unsigned char c) {
     if (flagChangeAlt == 0 && flagChangeF1 == 0) {
         flagChangeAlt = 1;
         flagChangeF1 = 1;
-        saveChar('\v');
+        // saveChar('\v');
+        saveChar(-1);
         return;
     } else if (c == 0xB8) {
         flagChangeAlt = 1;
diff --git a/Kernel/drivers/naiveConsole.c b/Kernel/drivers/naiveConsole.c
index 94b3280..d918efb 100644
--- a/Kernel/drivers/naiveConsole.c
+++ b/Kernel/drivers/naiveConsole.c
@@ -1,74 +1,76 @@
-#include <naiveConsole.h>
+// // This is a personal academic project. Dear PVS-Studio, please check it.
+// // PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
+// #include <naiveConsole.h>
 
-static uint32_t uintToBase(uint64_t value, char *buffer, uint32_t base);
+// static uint32_t uintToBase(uint64_t value, char *buffer, uint32_t base);
 
-static char buffer[64] = {'0'};
-static uint8_t *const video = (uint8_t *) 0xB8000;
-static uint8_t *currentVideo = (uint8_t *) 0xB8000;
-static const uint32_t width = 80;
-static const uint32_t height = 25;
+// static char buffer[64] = {'0'};
+// static uint8_t *const video = (uint8_t *) 0xB8000;
+// static uint8_t *currentVideo = (uint8_t *) 0xB8000;
+// static const uint32_t width = 80;
+// static const uint32_t height = 25;
 
-void ncPrint(const char *string) {
-    for (int i = 0; string[i] != 0; i++)
-        ncPrintChar(string[i]);
-}
+// void ncPrint(const char *string) {
+//     for (int i = 0; string[i] != 0; i++)
+//         ncPrintChar(string[i]);
+// }
 
-void ncPrintChar(char character) {
-    *currentVideo = character;
-    currentVideo += 2;
-}
+// void ncPrintChar(char character) {
+//     *currentVideo = character;
+//     currentVideo += 2;
+// }
 
-void ncNewline() {
-    do {
-        ncPrintChar(' ');
-    } while ((uint64_t)(currentVideo - video) % (width * 2) != 0);
-}
+// void ncNewline() {
+//     do {
+//         ncPrintChar(' ');
+//     } while ((uint64_t)(currentVideo - video) % (width * 2) != 0);
+// }
 
-void ncPrintDec(uint64_t value) {
-    ncPrintBase(value, 10);
-}
+// void ncPrintDec(uint64_t value) {
+//     ncPrintBase(value, 10);
+// }
 
-void ncPrintHex(uint64_t value) {
-    ncPrintBase(value, 16);
-}
+// void ncPrintHex(uint64_t value) {
+//     ncPrintBase(value, 16);
+// }
 
-void ncPrintBin(uint64_t value) {
-    ncPrintBase(value, 2);
-}
+// void ncPrintBin(uint64_t value) {
+//     ncPrintBase(value, 2);
+// }
 
-void ncPrintBase(uint64_t value, uint32_t base) {
-    uintToBase(value, buffer, base);
-    ncPrint(buffer);
-}
+// void ncPrintBase(uint64_t value, uint32_t base) {
+//     uintToBase(value, buffer, base);
+//     ncPrint(buffer);
+// }
 
-void ncClear() {
-    for (int i = 0; i < height * width; i++)
-        video[i * 2] = ' ';
-    currentVideo = video;
-}
+// void ncClear() {
+//     for (int i = 0; i < height * width; i++)
+//         video[i * 2] = ' ';
+//     currentVideo = video;
+// }
 
-static uint32_t uintToBase(uint64_t value, char *buffer, uint32_t base) {
-    char *p = buffer;
-    char *p1, *p2;
-    uint32_t digits = 0;
+// static uint32_t uintToBase(uint64_t value, char *buffer, uint32_t base) {
+//     char *p = buffer;
+//     char *p1, *p2;
+//     uint32_t digits = 0;
 
-    do {
-        uint32_t remainder = value % base;
-        *p++ = (remainder < 10) ? remainder + '0' : remainder + 'A' - 10;
-        digits++;
-    } while (value /= base);
+//     do {
+//         uint32_t remainder = value % base;
+//         *p++ = (remainder < 10) ? remainder + '0' : remainder + 'A' - 10;
+//         digits++;
+//     } while (value /= base);
 
-    *p = 0;
+//     *p = 0;
 
-    p1 = buffer;
-    p2 = p - 1;
-    while (p1 < p2) {
-        char tmp = *p1;
-        *p1 = *p2;
-        *p2 = tmp;
-        p1++;
-        p2--;
-    }
+//     p1 = buffer;
+//     p2 = p - 1;
+//     while (p1 < p2) {
+//         char tmp = *p1;
+//         *p1 = *p2;
+//         *p2 = tmp;
+//         p1++;
+//         p2--;
+//     }
 
-    return digits;
-}
+//     return digits;
+// }
diff --git a/Kernel/drivers/time.c b/Kernel/drivers/time.c
index d21d3c7..724061b 100755
--- a/Kernel/drivers/time.c
+++ b/Kernel/drivers/time.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "time.h"
 
 static unsigned long ticks = 0;
diff --git a/Kernel/drivers/video.c b/Kernel/drivers/video.c
index ef81b0e..40015d0 100644
--- a/Kernel/drivers/video.c
+++ b/Kernel/drivers/video.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "video.h"
 
 static uint8_t *const video = (uint8_t *) 0xB8000;
diff --git a/Kernel/interruptions/exceptions.c b/Kernel/interruptions/exceptions.c
index c7617b0..07e88ca 100755
--- a/Kernel/interruptions/exceptions.c
+++ b/Kernel/interruptions/exceptions.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "exceptions.h"
 
 static void zero_division();
diff --git a/Kernel/interruptions/idtLoader.c b/Kernel/interruptions/idtLoader.c
index db45ef8..00986dd 100755
--- a/Kernel/interruptions/idtLoader.c
+++ b/Kernel/interruptions/idtLoader.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "idtLoader.h"
 
 #pragma pack(push)
diff --git a/Kernel/interruptions/irqDispatcher.c b/Kernel/interruptions/irqDispatcher.c
index 925a929..9f9c698 100755
--- a/Kernel/interruptions/irqDispatcher.c
+++ b/Kernel/interruptions/irqDispatcher.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "irqDispatcher.h"
 
 void irqDispatcher(uint64_t irq) {
diff --git a/Kernel/interruptions/systemCalls.c b/Kernel/interruptions/systemCalls.c
index e5e5760..0ae8f15 100644
--- a/Kernel/interruptions/systemCalls.c
+++ b/Kernel/interruptions/systemCalls.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "systemCalls.h"
 
 uint64_t write(uint64_t fd, uint64_t buffer, uint64_t length) {
@@ -36,10 +38,9 @@ uint64_t read(uint64_t fd, uint64_t buffer, uint64_t length) {
                 blockIO();
             }
             readBytes++;
-            if (*bufferAux++ == '\v') {
+            if (*bufferAux++ == -1) {
                 break;
             }
-            // blockIO();
         }
     } else {
         while (length-- > 0) {
@@ -47,12 +48,11 @@ uint64_t read(uint64_t fd, uint64_t buffer, uint64_t length) {
             if (*bufferAux == 0)
                 break;
             readBytes++;
-            if (*bufferAux++ == '\v') {
+            if (*bufferAux++ == -1) {
                 break;
             }
         }
     }
-    // *bufferAux = 0;
 
     return readBytes;
 }
\ No newline at end of file
diff --git a/Kernel/interruptions/systemCallsDispatcher.c b/Kernel/interruptions/systemCallsDispatcher.c
index 7d28679..1287ee7 100644
--- a/Kernel/interruptions/systemCallsDispatcher.c
+++ b/Kernel/interruptions/systemCallsDispatcher.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "systemCallsDispatcher.h"
 
 uint64_t systemCallsDispatcher(uint64_t rdi, uint64_t rsi, uint64_t rdx, uint64_t rcx, uint64_t r8, uint64_t r9) {
@@ -14,7 +16,7 @@ uint64_t systemCallsDispatcher(uint64_t rdi, uint64_t rsi, uint64_t rdx, uint64_
             exitProcess();
             break;
         case 5:
-            return dumpMM();
+            return (uint64_t) dumpMM();
         case 6:
             return (uint64_t) processes();
         case 7:
@@ -51,11 +53,15 @@ uint64_t systemCallsDispatcher(uint64_t rdi, uint64_t rsi, uint64_t rdx, uint64_
         case 20:
             return unblock(rsi);
         case 21:
-            return wait();
+            wait();
+            break;
         case 22:
-            return pipes();
+            return (uint64_t) pipes();
         case 23:
             return quitCPU();
+        case 24:
+            closePipe(rsi);
+            break;
         default:
             return -1;
     }
diff --git a/Kernel/kernel.c b/Kernel/kernel.c
index 766ed2d..8ae36b6 100644
--- a/Kernel/kernel.c
+++ b/Kernel/kernel.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include <stdint.h>
 #include <lib.h>
 #include <moduleLoader.h>
diff --git a/Kernel/lib.c b/Kernel/lib.c
index 8f4cda0..61f3fe0 100644
--- a/Kernel/lib.c
+++ b/Kernel/lib.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "lib.h"
 
 void *memset(void *destination, int32_t c, uint64_t length) {
diff --git a/Kernel/moduleLoader.c b/Kernel/moduleLoader.c
index 293dea9..6f4f65a 100644
--- a/Kernel/moduleLoader.c
+++ b/Kernel/moduleLoader.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include <stdint.h>
 #include <lib.h>
 #include <moduleLoader.h>
diff --git a/Kernel/utils/include/pipeLib.h b/Kernel/utils/include/pipeLib.h
index f8db8f0..78bbc46 100644
--- a/Kernel/utils/include/pipeLib.h
+++ b/Kernel/utils/include/pipeLib.h
@@ -25,5 +25,6 @@ char openPipe(int * fds, char * name);
 void writePipe(int fd, char c);
 char readPipe(int fd);
 char * pipes();
+void closePipe(int fd);
 
 #endif
\ No newline at end of file
diff --git a/Kernel/utils/include/schedulerLib.h b/Kernel/utils/include/schedulerLib.h
index ec061e6..813c4b1 100644
--- a/Kernel/utils/include/schedulerLib.h
+++ b/Kernel/utils/include/schedulerLib.h
@@ -21,5 +21,7 @@ void sleep(int secs);
 char blockIO();
 void unblockIO();
 char getState(int pid);
+char isForeground();
+void wait();
 
 #endif
\ No newline at end of file
diff --git a/Kernel/utils/memManagerBuddy.c b/Kernel/utils/memManagerBuddy.c
index 8314529..f70d870 100644
--- a/Kernel/utils/memManagerBuddy.c
+++ b/Kernel/utils/memManagerBuddy.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #ifdef BUDDY
 
 // Basado en: https://github.com/cloudwu/buddy/blob/master/buddy.c
@@ -83,10 +85,9 @@ void * buddy_alloc(int s) {
 
 	int index = 0;
 	int level = 0;
-	int nextStep = 1;
 
 	while (index >= 0) {
-		nextStep = 1;
+		int nextStep = 1;
 		if (size == length) {
 			if (self.tree[index] == NODE_UNUSED) {
 				self.tree[index] = NODE_USED;
@@ -181,50 +182,12 @@ int buddy_free(void * pointer) {
 	}
 }
 
-// char *dumpMM() {
-// 	return NULL;
-// }
 uint64_t getSize(int level, int max_level) {
 	return (1 << (max_level - level)) * PAGE_SIZE;
 }
 
-/*
-void buddy_dumpMM(struct buddy * self, int index , int level) {
-    char buffer[10];
-	switch (self->tree[index]) {
-	case NODE_UNUSED:
-		printStringLen(15, "(_", 1);
-		printStringLen(15, itoa(getSize(level, self->level), buffer, 10), 0);
-		printStringLen(15, ":", 1);
-        printStringLen(15, itoa(level, buffer, 10), 10);
-		printStringLen(15, "_)", 1);
-		break;
-	case NODE_USED:
-		printStringLen(15, "(*", 1);
-		printStringLen(15, itoa(getSize(level, self->level), buffer, 10), 10);
-		printStringLen(15, ":", 1);
-        printStringLen(15, itoa(level, buffer, 10), 10);
-		printStringLen(15, "*)", 1);
-		break;
-	case NODE_FULL:
-		printStringLen(15, "[", 1);
-		buddy_dumpMM(self, index * 2 + 1, level + 1);
-		buddy_dumpMM(self, index * 2 + 2, level + 1);
-		printStringLen(15, "]", 1);
-		break;
-	default:
-		printStringLen(15, "(", 1);
-		buddy_dumpMM(self, index * 2 + 1, level + 1);
-		buddy_dumpMM(self, index * 2 + 2, level + 1);
-		printStringLen(15, ")", 1);
-		break;
-	}
-}
-*/
-
 #include <stddef.h>
 void buddy_dumpMM(int index , int level, uint64_t *size, uint64_t *used) {
-    char buffer[10];
 	switch (self.tree[index]) {
 	case NODE_UNUSED:
 		*size += getSize(level, self.level);
diff --git a/Kernel/utils/memManagerFRT4.c b/Kernel/utils/memManagerFRT4.c
index 622fbc3..6dc7d1d 100644
--- a/Kernel/utils/memManagerFRT4.c
+++ b/Kernel/utils/memManagerFRT4.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #ifndef BUDDY
 
 #include "memManagerFRT4.h"
@@ -118,7 +120,6 @@ void *pvPortMalloc(size_t xWantedSize) {
 }
 
 /*-----------------------------------------------------------*/
-
 void vPortFree(void *pv) {
     uint8_t *puc = (uint8_t *) pv;
     BlockLink_t *pxLink;
diff --git a/Kernel/utils/pipe.c b/Kernel/utils/pipe.c
index bed7832..a1a42d8 100644
--- a/Kernel/utils/pipe.c
+++ b/Kernel/utils/pipe.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "pipe.h"
 
 static int fdIndex = 2;
diff --git a/Kernel/utils/scheduler.c b/Kernel/utils/scheduler.c
index de04c33..e47a401 100644
--- a/Kernel/utils/scheduler.c
+++ b/Kernel/utils/scheduler.c
@@ -1,7 +1,9 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "scheduler.h"
 
 enum states {
-    READY = 0, DEAD, BLOCKED, BBCHILDREN, WAITING, BLOCKEDIO
+    READY = 0, DEAD, BLOCKED, WAITING, BLOCKEDIO
 };
 
 typedef struct processCDT {
@@ -18,6 +20,7 @@ typedef struct processCDT {
     int *fd;
     int children;
     char backWait;
+    uint64_t bPointer;
 } processCDT;
 
 typedef struct sleepCDT {
@@ -56,7 +59,6 @@ void removeProcess(processCDT *del, processCDT **first, processCDT **last) {
 uint64_t nextProcess(uint64_t currentRSP) {
     if (currentProcess != NULL)
         currentProcess->rsp = currentRSP;
-    processCDT *prev = currentProcess;
     if (currentProcess != NULL && currentProcess->state == READY &&
         currentProcess->executions == MAX_PRIORITY - currentProcess->priority) {
         currentProcess->executions = 0;
@@ -69,7 +71,6 @@ uint64_t nextProcess(uint64_t currentRSP) {
         } else if (currentProcess == firstBlockedIteration) {
             idleFlag = 1;
             unblock(IDLE_PID);
-            prev = currentProcess;
             currentProcess = currentProcess->next;
         } else if (currentProcess->state == DEAD) {
             processCDT *del = currentProcess;
@@ -80,7 +81,6 @@ uint64_t nextProcess(uint64_t currentRSP) {
                    currentProcess->state == BLOCKEDIO) {
             if (firstBlockedIteration == NULL)
                 firstBlockedIteration = currentProcess;
-            prev = currentProcess;
             currentProcess = currentProcess->next;
         }
     }
@@ -149,6 +149,7 @@ int enqueueProcess(void (*fn)(int, char **), char foreground, int argc, char *ar
     process->next = NULL;
     process->children = 0;
     process->backWait = 0;
+    process->bPointer = (uint64_t) auxi;
 
     process->rsp = _initialize_stack_frame(fn, rbp, argc, argv);
 
@@ -341,8 +342,8 @@ char kill(int pid) {
 
     vPortFree(del->fd);
     vPortFree(del->name);
-    void *auxi = del->rbp - STACK_SIZE;
-    vPortFree((void *) auxi);
+    // vPortFree((void *) ((uint64_t) del->rbp - STACK_SIZE));
+    vPortFree((void *) del->bPointer);
     del->state = DEAD;
 
     if (pid == currentProcess->pid) {
diff --git a/Kernel/utils/sem.c b/Kernel/utils/sem.c
index 9acd423..00f76ca 100644
--- a/Kernel/utils/sem.c
+++ b/Kernel/utils/sem.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "sem.h"
 
 static uint32_t semLock;
@@ -40,6 +42,8 @@ sem_t *semOpen(char *name, unsigned int value) {
     sem->name = pvPortMalloc(MAX_NAME);
     strcpy(sem->name, name);
     sem->value = value;
+    sem->entering = NULL;
+    sem->last = NULL;
     counter++;
 
     leave_region(&semLock);
diff --git a/Makefile b/Makefile
index b1f82c0..5c7d58e 100644
--- a/Makefile
+++ b/Makefile
@@ -37,4 +37,10 @@ clean:
 	cd Kernel; make clean
 	cd Userland; make clean
 
-.PHONY: bootloader image imageSpanish imageBuddy collections kernel kernelSpanish kernelBuddy userland all clean 
+test:
+	pvs-studio-analyzer trace -- make
+	pvs-studio-analyzer analyze
+	plog-converter -a '64:1,2,3;GA:1,2,3;OP:1,2,3' -t tasklist -o report.tasks PVS-Studio.log
+	cppcheck --quiet --enable=all --force --inconclusive .
+
+.PHONY: bootloader image imageSpanish imageBuddy collections kernel kernelSpanish kernelBuddy userland all clean test
\ No newline at end of file
diff --git a/Toolchain/ModulePacker/main.c b/Toolchain/ModulePacker/main.c
index 4edcc00..403c24b 100644
--- a/Toolchain/ModulePacker/main.c
+++ b/Toolchain/ModulePacker/main.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include <stdio.h>
 #include <unistd.h>
 #include <stdint.h>
diff --git a/Userland/SampleCodeModule/_loader.c b/Userland/SampleCodeModule/_loader.c
index 1a0c89d..e997f7c 100644
--- a/Userland/SampleCodeModule/_loader.c
+++ b/Userland/SampleCodeModule/_loader.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include <stdint.h>
 
 extern char bss;
diff --git a/Userland/SampleCodeModule/asm/libasm.asm b/Userland/SampleCodeModule/asm/libasm.asm
index 6c4b983..f7035b6 100644
--- a/Userland/SampleCodeModule/asm/libasm.asm
+++ b/Userland/SampleCodeModule/asm/libasm.asm
@@ -3,7 +3,7 @@ GLOBAL _getMem, sys_loadProcess
 GLOBAL raiseOpcodeExc
 GLOBAL _getRegs, sys_switchContext
 GLOBAL cpu_id, cpu_id_support
-GLOBAL sys_exit, sys_ps, sys_free, sys_malloc, sys_sem, sys_openPipe, sys_semClose
+GLOBAL sys_exit, sys_ps, sys_free, sys_malloc, sys_sem, sys_openPipe, sys_closePipe, sys_semClose
 GLOBAL sys_nice, sys_semWait, sys_semPost, sys_semOpen, sys_sleep, sys_kill, sys_getPid,
 GLOBAL sys_block, sys_unblock, sys_wait, sys_pipes, sys_quitCPU, sys_dumpMM
 
@@ -78,6 +78,24 @@ sys_openPipe:
     pop rbp
     ret
 
+sys_closePipe:
+    push rbp
+    mov rbp, rsp
+
+	push rdi
+	push rsi
+
+	mov rsi, rdi
+	mov rdi, 24
+	int 80h
+
+	pop rsi
+	pop rdi
+
+    mov rsp, rbp
+    pop rbp
+    ret
+
 sys_exit:
     push rbp
     mov rbp, rsp
diff --git a/Userland/SampleCodeModule/bottler/bottler.c b/Userland/SampleCodeModule/bottler/bottler.c
index 9039716..0a4df20 100644
--- a/Userland/SampleCodeModule/bottler/bottler.c
+++ b/Userland/SampleCodeModule/bottler/bottler.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "bottler.h"
 
 void bottler(int argc, char **argv) {
diff --git a/Userland/SampleCodeModule/include/system.h b/Userland/SampleCodeModule/include/system.h
index ba64e0b..125adff 100644
--- a/Userland/SampleCodeModule/include/system.h
+++ b/Userland/SampleCodeModule/include/system.h
@@ -36,5 +36,7 @@ char sys_unblock(int pid);
 void sys_quitCPU();
 void sys_wait();
 char * sys_dumpMM();
+char * sys_pipes();
+void sys_closePipe();
 
 #endif
\ No newline at end of file
diff --git a/Userland/SampleCodeModule/libc.c b/Userland/SampleCodeModule/libc.c
index 0bb6faa..d1c7798 100644
--- a/Userland/SampleCodeModule/libc.c
+++ b/Userland/SampleCodeModule/libc.c
@@ -154,7 +154,7 @@ char *itoa(int value, char *buffer, int base) {
 }
 
 int atoi(char *str, int length) {
-    int i = 0, sign = 1, val = 0, nbr = 0;
+    int i = 0, sign = 1, val = 0;
     while (str[i] != '\0') {
         if (str[i] == '-') {
             sign = -sign;
@@ -164,7 +164,7 @@ int atoi(char *str, int length) {
     }
     i = 0;
     while (str[i] >= '0' && str[i] <= '9' && str[i] != '\0') {
-        nbr = (int) (str[i] - '0');
+        int nbr = (int) (str[i] - '0');
         val = (val * 10) + nbr;
         i++;
     }
diff --git a/Userland/SampleCodeModule/sampleCodeModule.c b/Userland/SampleCodeModule/sampleCodeModule.c
index 28843ff..b53b1ba 100644
--- a/Userland/SampleCodeModule/sampleCodeModule.c
+++ b/Userland/SampleCodeModule/sampleCodeModule.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "shell/include/shell.h"
 #include "bottler/include/bottler.h"
diff --git a/Userland/SampleCodeModule/shell/commands/block.c b/Userland/SampleCodeModule/shell/commands/block.c
index e0b640f..243fda2 100644
--- a/Userland/SampleCodeModule/shell/commands/block.c
+++ b/Userland/SampleCodeModule/shell/commands/block.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "block.h"
 
 void block(int argc, char ** argv) {
diff --git a/Userland/SampleCodeModule/shell/commands/cat.c b/Userland/SampleCodeModule/shell/commands/cat.c
index a6e474c..da877bc 100644
--- a/Userland/SampleCodeModule/shell/commands/cat.c
+++ b/Userland/SampleCodeModule/shell/commands/cat.c
@@ -1,14 +1,20 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "cat.h"
 
 #define SIZE 1000
 
+void debug2() {
+    return;
+}
+
 void cat(int argc, char ** argv) {
     char c;
     int i = 0, j = 0, line = 0;
     char buffer[SIZE] = {0};
 
     while ((c = getChar()) > 0) {
-        if (i >= SIZE)
+        if (i > SIZE - 1)
             break;
         buffer[i++] = c;
         j++;
@@ -22,6 +28,10 @@ void cat(int argc, char ** argv) {
             buffer[--i] = 0;
         }
     }
+    buffer[i++] = c;
+    j++;
+
+    debug2();
     printStringLen(buffer + line, j);
 
     newline();
diff --git a/Userland/SampleCodeModule/shell/commands/clear.c b/Userland/SampleCodeModule/shell/commands/clear.c
index 6cf6a4f..5b8269b 100644
--- a/Userland/SampleCodeModule/shell/commands/clear.c
+++ b/Userland/SampleCodeModule/shell/commands/clear.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "shell.h"
 #include "clear.h"
diff --git a/Userland/SampleCodeModule/shell/commands/cpuid.c b/Userland/SampleCodeModule/shell/commands/cpuid.c
index 682c291..c9726d9 100644
--- a/Userland/SampleCodeModule/shell/commands/cpuid.c
+++ b/Userland/SampleCodeModule/shell/commands/cpuid.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "cpu_id.h"
 
 int cpu_id(int *, int *, int *, int *);
diff --git a/Userland/SampleCodeModule/shell/commands/excDiv.c b/Userland/SampleCodeModule/shell/commands/excDiv.c
index bc07068..44bd5c1 100644
--- a/Userland/SampleCodeModule/shell/commands/excDiv.c
+++ b/Userland/SampleCodeModule/shell/commands/excDiv.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "excDiv.h"
 
 void excdiv(int argc, char *argv[]) {
diff --git a/Userland/SampleCodeModule/shell/commands/excOP.c b/Userland/SampleCodeModule/shell/commands/excOP.c
index 0a4f297..3146047 100644
--- a/Userland/SampleCodeModule/shell/commands/excOP.c
+++ b/Userland/SampleCodeModule/shell/commands/excOP.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "excOP.h"
 
 void raiseOpcodeExc();
diff --git a/Userland/SampleCodeModule/shell/commands/filter.c b/Userland/SampleCodeModule/shell/commands/filter.c
index 186e16b..89f57cb 100644
--- a/Userland/SampleCodeModule/shell/commands/filter.c
+++ b/Userland/SampleCodeModule/shell/commands/filter.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "filter.h"
 
 #define SIZE 1000
@@ -28,6 +30,8 @@ void filter(int argc, char ** argv) {
             line = i;
         }
     }
+    buffer[i++] = c;
+    j++;
 
     printStringLen(buffer + line, j);
 
diff --git a/Userland/SampleCodeModule/shell/commands/help.c b/Userland/SampleCodeModule/shell/commands/help.c
index bd7527d..93943d6 100644
--- a/Userland/SampleCodeModule/shell/commands/help.c
+++ b/Userland/SampleCodeModule/shell/commands/help.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "shell.h"
 
diff --git a/Userland/SampleCodeModule/shell/commands/inforeg.c b/Userland/SampleCodeModule/shell/commands/inforeg.c
index d2c2154..5c618b2 100644
--- a/Userland/SampleCodeModule/shell/commands/inforeg.c
+++ b/Userland/SampleCodeModule/shell/commands/inforeg.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "inforeg.h"
 #include <stdint.h>
 #include "libc.h"
diff --git a/Userland/SampleCodeModule/shell/commands/kill.c b/Userland/SampleCodeModule/shell/commands/kill.c
index e3d436b..f51841c 100644
--- a/Userland/SampleCodeModule/shell/commands/kill.c
+++ b/Userland/SampleCodeModule/shell/commands/kill.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "kill.h"
 
 void kill(int argc, char ** argv) {
diff --git a/Userland/SampleCodeModule/shell/commands/loop.c b/Userland/SampleCodeModule/shell/commands/loop.c
index 98b3746..e5e4d28 100644
--- a/Userland/SampleCodeModule/shell/commands/loop.c
+++ b/Userland/SampleCodeModule/shell/commands/loop.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "loop.h"
 #define PUAN -1
 
diff --git a/Userland/SampleCodeModule/shell/commands/mem.c b/Userland/SampleCodeModule/shell/commands/mem.c
index bd4d8de..44b226e 100644
--- a/Userland/SampleCodeModule/shell/commands/mem.c
+++ b/Userland/SampleCodeModule/shell/commands/mem.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "shell.h"
 
diff --git a/Userland/SampleCodeModule/shell/commands/nice.c b/Userland/SampleCodeModule/shell/commands/nice.c
index 838aa6b..6caee03 100644
--- a/Userland/SampleCodeModule/shell/commands/nice.c
+++ b/Userland/SampleCodeModule/shell/commands/nice.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "nice.h"
 
 void nice(int argc, char ** argv) {
diff --git a/Userland/SampleCodeModule/shell/commands/phylo.c b/Userland/SampleCodeModule/shell/commands/phylo.c
index aada59a..58d35db 100644
--- a/Userland/SampleCodeModule/shell/commands/phylo.c
+++ b/Userland/SampleCodeModule/shell/commands/phylo.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "phyloLib.h"
 
 philosopher_t *firstPhil;
@@ -120,7 +122,6 @@ void phylo(int argc, char ** argv) {
     philosopher_t * left = NULL;
     for (int i = 0; i < philoCount; i++) {
         philosopher_t * phil = sys_malloc(sizeof(philosopher_t));
-        phil->debug = i;
         phil->argv = sys_malloc(sizeof(char *) * 2);
         phil->buffer = sys_malloc(20);
         phil->state = THINKING;
diff --git a/Userland/SampleCodeModule/shell/commands/pipe.c b/Userland/SampleCodeModule/shell/commands/pipe.c
index 640fb6a..98c0dbc 100644
--- a/Userland/SampleCodeModule/shell/commands/pipe.c
+++ b/Userland/SampleCodeModule/shell/commands/pipe.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "pipes.h"
 
 void pipe(int argc, char *argv[]) {
diff --git a/Userland/SampleCodeModule/shell/commands/printmem.c b/Userland/SampleCodeModule/shell/commands/printmem.c
index 7115cf8..3ba4f96 100644
--- a/Userland/SampleCodeModule/shell/commands/printmem.c
+++ b/Userland/SampleCodeModule/shell/commands/printmem.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "inforeg.h"
 #include <stdint.h>
 #include "shell.h"
diff --git a/Userland/SampleCodeModule/shell/commands/ps.c b/Userland/SampleCodeModule/shell/commands/ps.c
index 45a1c2c..31631f2 100644
--- a/Userland/SampleCodeModule/shell/commands/ps.c
+++ b/Userland/SampleCodeModule/shell/commands/ps.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "shell.h"
 
diff --git a/Userland/SampleCodeModule/shell/commands/quadratic.c b/Userland/SampleCodeModule/shell/commands/quadratic.c
index f675997..8e5da5e 100644
--- a/Userland/SampleCodeModule/shell/commands/quadratic.c
+++ b/Userland/SampleCodeModule/shell/commands/quadratic.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "quadratic.h"
 #include "shell.h"
diff --git a/Userland/SampleCodeModule/shell/commands/sems.c b/Userland/SampleCodeModule/shell/commands/sems.c
index d41d80d..99302cf 100644
--- a/Userland/SampleCodeModule/shell/commands/sems.c
+++ b/Userland/SampleCodeModule/shell/commands/sems.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "libc.h"
 #include "shell.h"
 
diff --git a/Userland/SampleCodeModule/shell/commands/test_mm.c b/Userland/SampleCodeModule/shell/commands/test_mm.c
index 0f25d89..d358dff 100644
--- a/Userland/SampleCodeModule/shell/commands/test_mm.c
+++ b/Userland/SampleCodeModule/shell/commands/test_mm.c
@@ -1,5 +1,9 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "test_mm.h"
 
+void *memset(void *destination, int32_t c, uint64_t length);
+
 typedef struct MM_rq{
   void *address;
   uint32_t size;
diff --git a/Userland/SampleCodeModule/shell/commands/test_prio.c b/Userland/SampleCodeModule/shell/commands/test_prio.c
index ee05ae3..cf57fe4 100644
--- a/Userland/SampleCodeModule/shell/commands/test_prio.c
+++ b/Userland/SampleCodeModule/shell/commands/test_prio.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "test_prio.h"
 
 #define MINOR_WAIT 10000000                               // TODO: To prevent a process from flooding the screen
diff --git a/Userland/SampleCodeModule/shell/commands/test_processes.c b/Userland/SampleCodeModule/shell/commands/test_processes.c
index b4374af..a29d390 100644
--- a/Userland/SampleCodeModule/shell/commands/test_processes.c
+++ b/Userland/SampleCodeModule/shell/commands/test_processes.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "test_processes.h"
 #include "ps.h"
 
@@ -16,10 +18,6 @@ void endless_loop(){
   sys_exit();
 }
 
-void debugPSA() {
-  return;
-}
-
 void test_processes(int argc, char ** argv){
   p_rq p_rqs[MAX_PROCESSES];
   uint8_t rq;
diff --git a/Userland/SampleCodeModule/shell/commands/test_sync.c b/Userland/SampleCodeModule/shell/commands/test_sync.c
index 167215c..a5a3833 100644
--- a/Userland/SampleCodeModule/shell/commands/test_sync.c
+++ b/Userland/SampleCodeModule/shell/commands/test_sync.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "test_sync.h"
 
 #define TOTAL_PAIR_PROCESSES 2
@@ -25,11 +27,10 @@ void inc(int argc, char ** argv) {
   uint64_t N = atoi(argv[3], 10);
   uint64_t i;
 
-  char buff[10] = {0};
   for (i = 0; i < N; i++){
-    if (semDir) sys_semWait(semDir);
+    if (semDir) sys_semWait((sem_t *) semDir);
     slowInc(&global, value);
-    if (semDir) sys_semPost(semDir);
+    if (semDir) sys_semPost((sem_t *) semDir);
   }
 
   char buffer[10] = {0};
@@ -48,7 +49,7 @@ void test_sync(int argc, char ** argv){
 
   sem_t * sem = sys_semOpen(SEM_ID, 1);
   char semDir[10] = {0};
-  itoa((int) sem, semDir, 10);
+  itoa((uint64_t) sem, semDir, 10);
 
   for(i = 0; i < TOTAL_PAIR_PROCESSES; i++){
     char * argv1[] = {"inc", semDir, "1", ITERS};
diff --git a/Userland/SampleCodeModule/shell/commands/test_util.c b/Userland/SampleCodeModule/shell/commands/test_util.c
index 495f254..f870bf5 100644
--- a/Userland/SampleCodeModule/shell/commands/test_util.c
+++ b/Userland/SampleCodeModule/shell/commands/test_util.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include <stdint.h>
 #include "test_util.h"
 
diff --git a/Userland/SampleCodeModule/shell/commands/time.c b/Userland/SampleCodeModule/shell/commands/time.c
index 5e1e89f..0930324 100644
--- a/Userland/SampleCodeModule/shell/commands/time.c
+++ b/Userland/SampleCodeModule/shell/commands/time.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "time.h"
 #include "libc.h"
 #include "shell.h"
diff --git a/Userland/SampleCodeModule/shell/commands/unblock.c b/Userland/SampleCodeModule/shell/commands/unblock.c
index 8fd5753..4e503bc 100644
--- a/Userland/SampleCodeModule/shell/commands/unblock.c
+++ b/Userland/SampleCodeModule/shell/commands/unblock.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "unblock.h"
 
 void unblock(int argc, char ** argv) {
diff --git a/Userland/SampleCodeModule/shell/commands/wc.c b/Userland/SampleCodeModule/shell/commands/wc.c
index 72d65bd..ae5d5f3 100644
--- a/Userland/SampleCodeModule/shell/commands/wc.c
+++ b/Userland/SampleCodeModule/shell/commands/wc.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "wc.h"
 
 #define SIZE 1000
@@ -12,6 +14,7 @@ void wc(int argc, char ** argv) {
         if (c == '\n')
             i++;
     }
+    buffer[i++] = c;
 
     printStringLen(gtoa(i, buffer, 10, SIZE), SIZE);
     newline();
diff --git a/Userland/SampleCodeModule/shell/include/phyloLib.h b/Userland/SampleCodeModule/shell/include/phyloLib.h
index 84f2db3..ed48318 100644
--- a/Userland/SampleCodeModule/shell/include/phyloLib.h
+++ b/Userland/SampleCodeModule/shell/include/phyloLib.h
@@ -17,7 +17,6 @@ int * state;
 typedef enum states {EATING = 0, HUNGRY, THINKING} states;
 
 typedef struct philosopher_t {
-    int debug;
     char ** argv;
     char * buffer;
     sem_t * sem;
@@ -29,5 +28,7 @@ typedef struct philosopher_t {
 } philosopher_t;
 
 void philosopher(int argc, char ** argv);
+void freePhilo(philosopher_t * phil);
+void end();
 
 #endif
\ No newline at end of file
diff --git a/Userland/SampleCodeModule/shell/shell.c b/Userland/SampleCodeModule/shell/shell.c
index 9f11d3e..a4fa86d 100644
--- a/Userland/SampleCodeModule/shell/shell.c
+++ b/Userland/SampleCodeModule/shell/shell.c
@@ -1,3 +1,5 @@
+// This is a personal academic project. Dear PVS-Studio, please check it.
+// PVS-Studio Static Code Analyzer for C, C++ and C#: http://www.viva64.com
 #include "shell.h"
 
 cmd_t commands[] = {
@@ -144,9 +146,11 @@ void processInput(char *input) {
 
         if (commands[comm0].isForeground) {
             sys_wait();
-            sys_free(argv0);
-            if (pipe == -1)
-                sys_free(argv1);
+        }
+        sys_free(argv0);
+        if (pipe != -1) {
+            sys_free(argv1);
+            sys_closePipe(fd1[1]);
         }
     }
 
diff --git a/build.sh b/build.sh
index 90d1360..64a2c0d 100755
--- a/build.sh
+++ b/build.sh
@@ -11,9 +11,11 @@ cd ..
 
 if [ $# -eq 1 ]; then
     if [ $1 = "buddy" ]; then
-	make buddy
+	    make buddy
+    elif [ $1 = "spanish" ]; then
+        make spanish
     else
-	echo "Do you want to compile with the buddy memory manager? if so, you must pass buddy as argument"
+	    echo "Do you want to compile with the buddy memory manager? if so, you must pass buddy as argument"
     fi
 elif [ $# -eq 0 ]; then
     make all
diff --git a/flags.txt b/flags.txt
index 898febc..4e9b0ab 100644
--- a/flags.txt
+++ b/flags.txt
@@ -1 +1 @@
-be
+ke