Почему malloc не может работать с strcpy?

После цикла while выполните:

copy[j] = '\0';

чтобы NULL завершить вашу строку; таким образом он может работать с методами, происходящими из <string.h>, которые предполагают, что строка завершается нулем.

PS: одно из предупреждений, которое вы должны увидеть, касается того, что вы не должны возвращать copy в своей функции в любом случае, потому что теперь, если условие оператора if неверно, ваша функция не вернет что-то действительное, поэтому добавьте это:

return copy;

в конце вашей функции (которая теперь исправлена ​​вашим редактированием).

Кроме этого, единственное предупреждение, которое вы все равно должны получить, касается неиспользуемых аргументов main(), ничего больше:

prog.c: In function 'main':
prog.c:32:14: warning: unused parameter 'argc' [-Wunused-parameter]
 int main(int argc, char * argv[]){
              ^~~~
prog.c:32:27: warning: unused parameter 'argv' [-Wunused-parameter]
 int main(int argc, char * argv[]){
                           ^~~~

При копировании байтов из str в copy вы не добавляете завершающий нулевой байт в конце. В результате strcmp читает скопированные символы в унифицированную память, возможно, за конец выделенного блока памяти. Это вызывает неопределенное поведение.

После цикла while добавьте завершающий нулевой байт в copy.

Кроме того, вы никогда не вернете значение, если блок if в конце имеет значение false. Для этого вам нужно что-то вернуть, возможно, скопированную строку.

char * removeChar(char * str, char c){
    int len = strlen(str);
    int i = 0;
    int j = 0;
    char * copy = malloc(sizeof(char) * (len + 1));
    while(i < len){
        if(str[i] != c){
            copy[j] = str[i];
            j++;
            i++;
        }else{
            i++;
        }
    }
    //  add terminating null byte
    copy[j] = '\0';

    if(strcmp(copy, str) != 0){
       strcpy(str,copy);
    }
    // always return copy
    return copy;
}

Вы никогда не инициализировали ввод, и некоторые компиляторы не замечают, что значение никогда не используется перед строкой

input = removeChar(str, 'g');

в вашем коде. Поэтому они выдают диагностику просто для уверенности.

strcpy(str, copy)

застревает в вашем коде, так как копия никогда не получает закрывающий 0 байт и поэтому зависит от недетерминированного содержимого вашей памяти в момент выделения резервной копии памяти, как долго будет работать strcpy и получите ли вы в конечном итоге SIGSEGV (или аналогичный).

strcpy будет зацикливаться, пока не найдет 0 байт в вашей памяти.

Для начала, чтобы удалить символ из строки, нет необходимости динамически создавать массив символов, а затем копировать этот массив в исходную строку.

Либо вы должны написать функцию, которая действительно удаляет указанный символ из строки, либо функцию, которая создает новую строку на основе исходной строки, исключая указанный символ.

Это просто плохой дизайн, который только смущает пользователей. То есть функция слишком сложна и использует избыточные функции, такие как malloc, strlen, strcmp и strcpy. И на самом деле у него есть побочный эффект, который не очевиден. Кроме того, используется некорректный тип int для длины строки вместо типа size_t.

Что касается реализации вашей функции, то вы забыли добавить завершающий ноль '\ 0' к строке, построенной в динамически выделенном массиве.

Если вы действительно хотите удалить символ из строки, функция может выглядеть так, как показано в демонстрационной программе.

#include <stdio.h>

char * remove_char(char *s, char c)
{
    char *p = s;

    while (*p && *p != c) ++p;

    for ( char *q = p; *p++; )
    {
        if (*p != c) *q++ = *p;
    }

    return s;
}

int main( void )
{
    char str[] = "Input string";

    puts(str);
    puts(remove_char(str, 'g'));

    return 0;
}

Вывод программы

Input string
Input strin

Если вы изучаете функцию malloc и хотите ее использовать, вы в любом случае должны попытаться реализовать правильный дизайн.

Чтобы использовать malloc, вы можете написать функцию, которая создает новую строку на основе исходной строки, исключая указанный символ. Например

#include <stdio.h>
#include <stdlib.h>

char * remove_copy_char(const char *s, char c)
{
    size_t n = 0;

    for (const char *p = s; *p; ++p)
    {
        if (*p != c) ++n;
    }

    char *result = malloc(n + 1);

    if (result)
    {
        char *q = result;

        for (; *s; ++s)
        {
            if (*s != c) *q++ = *s;
        }

        *q = '\0';
    }

    return result;
}

int main( void )
{
    char *str = "Input string";

    puts(str);

    char *p = remove_copy_char(str, 'g');

    if ( p ) puts(p );

    free(p);

    return 0;
}

Вывод программы будет таким же, как указано выше.

Input string
Input strin

Обратите внимание на объявление функции

char * remove_copy_char(const char *s, char c);
                        ^^^^^^

В этом случае исходная строка может быть строковым литералом.

char *str = "Input string";