strcat() и fgets() в C не работают

Для вашего fgets() вы правильно указали:

9 элементов, потому что я читал, что вам нужно учитывать нулевой завершающий символ в вашей сумме.

Но в ваших объявлениях переменных вы забыли об этом:

char userName[8]; char password[8];

Сделайте их char ....[9], и ваша программа больше не должна прерываться.

Но, как упомянул @chux:

если вы читаете целые строки текста, вам также придется иметь дело с символом новой строки, что делает ваш код примерно таким:

char userName[10]; char password[10];
...
fgets( userName, sizeof userName, stdin );
if( userName[strlen(userName)-1] == '\n' ) {
    //truncate new line character
    userName[strlen(userName)-1] = '\0';
}

... и все равно у вас возникнет проблема, если кто-то введет имя или пароль длиной более 8 символов. Итак, я бы предложил:

#define L_USER 8

char userName[256]; char password[256];
...
fgets( userName, sizeof userName, stdin );
if( strlen(userName) > L_USER ) {
    //truncate name if too long
    userName[L_USER] = '\0';
}    
if( userName[strlen(userName)-1] == '\n' ) {
    //truncate new line character
    userName[strlen(userName)-1] = '\0';
}

Это радикальная переработка, но она также эффективна. Это больше похоже на то, что я бы использовал:

#include <assert.h>
#include <stdio.h>
#include <string.h>

static int prompt_for_data(const char *prompt, const char *what, char *buffer, size_t buflen)
{
    char line[256];
    assert(prompt != 0 && *prompt != '\0');
    assert(what != 0 && *what != '\0');
    assert(buffer != 0);
    assert(buflen > 1 && buflen < sizeof(line));

    printf("%s", prompt);
    if (fgets(line, sizeof(line), stdin) == 0)
    {
        fprintf(stderr, "EOF\n");
        return EOF;
    }
    size_t len = strlen(line);
    if (len == 0)
    {
        fprintf(stderr, "Zero length string? You can't use Control-@ in a %s\n",
                what);
        return EOF;
    }
    if (line[len - 1] != '\n')
    {
        fprintf(stderr, "Line far too long (%zu bytes)\n", len);
        return EOF;
    }
    line[--len] = '\0';
    /* Optionally chop leading and trailing white space - omitted */
    if (len > buflen - 1)
    {
        fprintf(stderr, "Maximum length of %s is %zu (you entered %zu characters)\n",
                what, buflen - 1, len);
        return EOF;
    }
    if (len == 0)
    {
        fprintf(stderr, "You're supposed to enter a value for the %s\n", what);
        return EOF;
    }
    strcpy(buffer, line);
    return 0;
}

int main(void)
{
    char userName[9];
    char password[9];

    if (prompt_for_data("Enter username: ", "username", userName, sizeof(userName)) == 0 &&
        prompt_for_data("Enter password: ", "password", password, sizeof(password)) == 0)
    {
        char str[150];
        snprintf(str, sizeof(str),
                 "INSERT INTO USERS (ID, USERNAME, PASSWORD, IP) VALUES"
                 "(%d, '%s', '%s', '%s');",
                 1, userName, password, "120.132.12.1");
        puts(str);
    }
    return 0;
}

Последовательность «запрашивать значение, читать строку, проверять ее длину, копировать в нужную переменную» заключена в функции prompt_for_data(). Затем он используется дважды (но я, вероятно, написал бы его как функцию, даже если бы он вызывался только один раз), чтобы получить имя пользователя и пароль. Затем, вместо использования strcat(), я использую snprintf() для создания выходной строки.

Вы также должны отметить, что если вы хотите изменить размер вашего имени пользователя и пароля с причудливых архаичных 8 символов плюс завершающий нуль на 32, единственные строки кода, которые изменяются, — это объявления userName и password. Это потому, что я осторожно использовал sizeof(userName) или sizeof(password), когда мне нужно было описать размер. Аналогично с другими переменными; например, строка fgets() не изменится, если я изменю размер переменной line. Обычно хорошей практикой является минимизация количества необходимых изменений при изменении размера переменной.

Обратите внимание, что у вас все еще есть проблемы с внедрением SQL. Если я наберу "O'Reilly" в качестве имени пользователя, SQL будет недействительным. Помните Little Bobby Tables! Одна хорошая вещь о коротких именах пользователей; это усложняет внедрение SQL.

Примеры запусков

$ ./up
Enter username: beckah12
Enter password: beckah12
INSERT INTO USERS (ID, USERNAME, PASSWORD, IP) VALUES(1, 'beckah12', 'beckah12', '120.132.12.1');
$ ./up
Enter username: absolom12
Maximum length of username is 8 (you entered 9 characters)
$ ./up
Enter username: beckah12
Enter password: absolom12
Maximum length of password is 8 (you entered 9 characters)
$ ./up
Enter username: 
You're supposed to enter a value for the username
$ ./up
Enter username: beckah12
Enter password: 
You're supposed to enter a value for the password
$ ./up
Enter username: EOF
$ ./up
Enter username: beckah12
Enter password: EOF
$ ./up
Enter username: nnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnnn
Line far too long (255 bytes)
$

Изменить линии

char userName[8]  to  char userName[9]

char password[8]  to  char password[9]

потому что это то, что вы указали в fgets(userName, 9, stdin); et. др..