Как исправить "назначение делает указатель из целого числа без приведения [-Wint-преобразование]"?C Unix программа - PullRequest
0 голосов
/ 19 июня 2019

Я пишу эти 2 программы, но получаю это предупреждение: «присваивание делает указатель из целого числа без приведения [-Wint-преобразование]».

Я пытаюсь скомпилировать эти программы на другие машиныНо я получаю ту же проблему.Что я могу сделать?

//PROGRAM 1  (PRODUCTOR)

#include <errno.h>
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <sys/sem.h>
#include <stdio.h>

#define SHMEMSIZE 4096

int main()
{

    key_t key_mem=6868;
    key_t key_sem=3232;
    int id_mem;
    int id_sem;
    void *addr_mem;
    struct sembuf param[1];

    if(id_mem = shmget(key_mem,SHMEMSIZE,0) == -1)
    {
        printf("Errore1\n");
        return -1;
    }


    if(addr_mem =shmat(id_mem,NULL,0) == (void *)-1)
    {
        printf("Errore2\n");
        return -1;
    }

    if(id_sem = semget(key_sem,2,0)== -1)
    {   
        perror("Errore3\n");
        shmdt(addr_mem);
        return -1;
    }

    while(1)
    {
        param[0].sem_num=1;
        param[0].sem_op=-1;
        param[0].sem_flg=0;

        if(semop(id_sem,param,1)==-1)
        {
            printf("Errore6\n");
            return -1;
        }
        printf("Scrivi messaggio: ");
        if(scanf("%[^\n]", (char *)addr_mem) == 0)
             *((char *)addr_mem) = '\0';
        getc(stdin);

        param[0].sem_num=0;
        param[0].sem_op=1;
        param[0].sem_flg=0;
        if(semop(id_sem,param,1)==-1)
        {
            printf("Errore7\n");
            return -1;
        }
    }

    shmdt(addr_mem);

}






_____________________________________________________

//PROGRAM 2 (CONSUMER)
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <sys/sem.h>
#include <stdio.h>
#define SHMEMSIZE 4096

union semun 
{
    int val;
    struct semid_ds *buf;
    unsigned short *array;
    struct seminfo *__buf;
};

int main()
{

    key_t key_mem=6868;
    key_t key_sem=3232;
    int id_mem;
    int id_sem;
    void *addr_mem;

    union semun arg;
    struct sembuf param[1];

    if(id_mem = shmget(key_mem,SHMEMSIZE,IPC_CREAT|0666) == -1)
    {
        printf("Errore1");
        return -1;
    }

    if(addr_mem =shmat(id_mem,NULL,0) == (void *)-1)
    {
        printf("Errore2");
        shmctl(id_mem, IPC_RMID, NULL);
        return -1;
    }

    if(id_sem = semget(key_sem,2,IPC_CREAT|0666)== -1)
    {   
        printf("Errore3");
        shmctl(id_mem, IPC_RMID, NULL);
        shmdt(addr_mem);
        return -1;
    }

    arg.val=0;
    if(semctl(id_sem,0,SETVAL,arg)==-1)
    {
        printf("Errore4");
        semctl(id_sem, -1, IPC_RMID,arg);
        shmctl(id_sem, IPC_RMID, NULL);
        shmdt(addr_mem);
        return -1;
    }
    arg.val=1;
    if(semctl(id_sem,1,SETVAL,arg)==-1)
    {
        printf("Errore5");
        semctl(id_sem, -1, IPC_RMID, arg);
        shmctl(id_sem, IPC_RMID, NULL);
        shmdt(addr_mem);
        return -1;
    }

    while(1)
    {
        param[0].sem_num=0;
        param[0].sem_op=-1;
        param[0].sem_flg=0;

        if(semop(id_sem,param,1)==-1)
        {
            printf("Errore6");
            return -1;
        }

        printf("Il messaggio scritto è: %s\n",(char *)addr_mem);
        param[0].sem_num=1;
        param[0].sem_op=1;
        param[0].sem_flg=0;
        if(semop(id_sem,param,1)==-1)
        {
            printf("Errore7");
            return -1;
        }
    }

}

Моя программа должна просто разрешить процессу производителя писать в общей памяти, а процессу потребителя - общую память.

Сначала я получаю это "предупреждение" ипосле этого, если я пытаюсь запустить продюсера, я получаю «Ошибка сегментации».Я абсолютно не могу понять, что происходит.

Я полагаю, что общая память на самом деле не создается, на самом деле, если я удаляю строку * ((char *) addr_mem) = '0';, я не получаю "Ошибка сегментации", и этоПонятно, что «Сканф» ничего не пишет.

1 Ответ

4 голосов
/ 19 июня 2019

IMO if(id_mem = shmget(key_mem,SHMEMSIZE,IPC_CREAT|0666) == -1) - очень плохой стиль, именно для вашей проблемы. Вы забыли добавить парантезы вокруг задания. Вы сравниваете результат shmget с -1 и присваиваете этот результат id_mem.

Измените его на if((id_mem = shmget(key_mem,SHMEMSIZE,IPC_CREAT|0666)) == -1) или даже лучше

id_mem = shmget(key_mem,SHMEMSIZE,IPC_CREAT|0666);
if (id_mem == -1)

Следующие if s имеют ту же проблему.

...