From 6781ac89adf9e8442d08bf1480e37ff25aff32a1 Mon Sep 17 00:00:00 2001
From: Georg Reisinger <georg@dergeorg.at>
Date: Wed, 30 Sep 2020 15:11:32 -0700
Subject: [PATCH] GetUserMessages bug fix

---
 myserver.c | 30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/myserver.c b/myserver.c
index a6a0dd9..2be6393 100755
--- a/myserver.c
+++ b/myserver.c
@@ -49,10 +49,13 @@ struct msgNumberArr getUserMessages(char username[1024])
 {
    FILE *filePointer;
    char filename[1024];
+   if('\n' == username[strlen(username) - 1]){
+      username[strlen(username) - 1] = '\0';
+   }
    strcpy(filename, username);
    strcat(filename, ".bin");
-   filePointer = fopen(filename, "wb");
-   printf("start getUserMessages() Filename: %s, Username: %s\n", filename, username);
+   filePointer = fopen(filename, "rb");
+   printf("\n\nstart getUserMessages() Filename: %s, Username: %s\n\n", filename, username);
    if (!filePointer)
    {
       printf("\x1B[31User Message Speicher konnte nicht gelesen werden!\x1B[37m\n");
@@ -62,17 +65,18 @@ struct msgNumberArr getUserMessages(char username[1024])
    }
    int i = 0;
    struct msgNumberArr msgNumbersStruct;
-   
+
+
    //int msgNumberArr[1024];
-   
+
    //printf("Vorm while\n");
    //while(!feof(filePointer))
    //while (fread(&msgNumbers, sizeof(struct msgNumber), 1, filePointer))
    //{
-      //printf("Im while\n");
-   
-   fread(&msgNumbersStruct, sizeof(struct msgNumberArr),1, filePointer);
-   
+   //printf("Im while\n");
+
+   fread(&msgNumbersStruct, sizeof(struct msgNumberArr), 1, filePointer);
+   printf("\nmsgNumberSrtruct lenght: %d\n", msgNumbersStruct.lenght);
    //}
    //printf("Nach dem while\n");
    fclose(filePointer);
@@ -81,6 +85,10 @@ struct msgNumberArr getUserMessages(char username[1024])
 
 int addUserMessage(char username[1024], int msgNumber)
 {
+   if('\n' == username[strlen(username) - 1]){
+      username[strlen(username) - 1] = '\0';
+   }
+   //username[strlen(username)-1] = "\0";
    printf("addUserMessage(), username: %s, msgNumber: %d\n", username, msgNumber);
    int msgNumbers[1024];
    struct msgNumberArr msgNumbersStruct = getUserMessages(username);
@@ -108,8 +116,8 @@ int addUserMessage(char username[1024], int msgNumber)
       newNumbers[0] = msgNumber;
       for (int i = 1; i <= nMsgs; i++)
       {
-         printf("addUserMessage() msg number old: %d", msgNumbers[i-1]);
-         newNumbers[i] = msgNumbers[i-1];
+         printf("addUserMessage() msg number old: %d", msgNumbers[i - 1]);
+         newNumbers[i] = msgNumbers[i - 1];
       }
       printf("Vor dem writeUsersMessages() username: %s", username);
       result = writeUsersMessages(username, newNumbers, nMsgs + 1);
@@ -415,10 +423,8 @@ int main(int argc, char **argv)
                   printf("->LIST->Username wurde eingegeben:\x1B[33m %s\x1B[37m\n", bufferString);
                   char *username;
                   username = strdup(bufferString);
-                  username[strlen(username)-1] = '\0';
                   printf("USERNAME: %s ENDE", username);
 
-
                   struct msgNumberArr msgNumbersStruct = getUserMessages(username);
                   printf("msgnumberstruct lenght: %d", msgNumbersStruct.lenght);
                   int msgNumbers[1024];