Bonjour lorsque j'utilise une fonction permettant de lire un fichier, le copier dans un string et retourner ce string, visual studio community me donne un Warning :
"Warning C6386 Buffer overrun while writing to 'buffer': the writable size is 'size+1' bytes, but '4294967295' bytes might be written."

Voici ma fonction :

static char* readShader(const char* filepath)
{
    FILE* file;
    size_t read_bytes = 0;

    file = fopen(filepath, "r");
    if (file == NULL)
    {
        printf("Could not open shader file");
        return NULL;
    }
    fseek(file, 0L, SEEK_END);
    const int size = ftell(file);
    fseek(file, 0L, SEEK_SET);

    char* buffer = (char*)malloc(size + 1);
    if (buffer == NULL)
    {
        printf("Could not allocate memory");
        return NULL;
    }
    read_bytes = fread(buffer, sizeof(char), size, file);
    buffer[read_bytes] = '\0';
    fclose(file);
    return buffer;
}

La fonction ftell retourne un long, fread prend un size_t qui, selon l'implémentation, est souvent un simple int. Si tu obtiens -1L (qui est le code de retour d'erreur de ftell) en retour de ftell, tu te retrouves avec un unsigned long !

Pour résoudre ce problème, vérifie la valeur de retour de ftell et assure-toi qu'elle ne vaut pas -1L, puis, lors de l'appel à fread, convertis la variable size en size_t.

Je n'ai plus le warning en changeant le type de size de int a size_t :

static char* readShader(const char* filepath)
{
    FILE* file;
    size_t read_bytes = 0;

    file = fopen(filepath, "r");
    if (file == NULL)
    {
        printf("Could not open shader file");
        return NULL;
    }
    fseek(file, 0L, SEEK_END);
    const size_t size = ftell(file);  // ici je change le type de size de int à size_t
    fseek(file, 0L, SEEK_SET);

    char* buffer = (char*)malloc(size + 1);
    if (buffer == NULL)
    {
        printf("Could not allocate memory");
        return NULL;
    }
    read_bytes = fread(buffer, sizeof(char), size, file);
    buffer[read_bytes] = '\0';
    fclose(file);
    return buffer;
}

Cepandant ceci ne fonction pas en typecast size en size_t ;

static char* readShader(const char* filepath)
{
    FILE* file;
    size_t read_bytes = 0;

    file = fopen(filepath, "r");
    if (file == NULL)
    {
        printf("Could not open shader file");
        return NULL;
    }
    fseek(file, 0L, SEEK_END);
    const int size = ftell(file);
    fseek(file, 0L, SEEK_SET);

    char* buffer = (char*)malloc(size + 1);
    if (buffer == NULL)
    {
        printf("Could not allocate memory");
        return NULL;
    }
    read_bytes = fread(buffer, sizeof(char), (size_t)size, file); // ici je typecast size en size_t
    buffer[read_bytes] = '\0';
    fclose(file);
    return buffer;
}

Tu veux dire quoi par "ça ne fonctionne pas" ?

ton code a l'air bon.
A mon avis c'est un warning qui indique que tu peux avoir un soucis de debordement, mais il ne veut pas dire que tu va en avoir un.

pour que ce soit plus propre je mettrais dans le fopen "rb" (binary), pour lire exactement ce qu'il y a dans le fichier (sinon en mode ascii, certains caracteres peuvent etre reinteprettés, comme les CR en 1 ou 2 bytes)