La Mémoire Partagée Avec Deux Processus En C?

Ok, mieux recueillir dans une réponse à la place...

Il y a plusieurs problèmes avec vous programme. Si vous activez les mises en garde lors de la construction (j'utilise -Wall -Wextra) beaucoup d'entre eux seront tout à fait évident.

Les deux premiers problèmes que j'ai déjà mentionné dans mes commentaires, mais je l'explique ici:

1. Le premier est l'appel à wait(). Il n'y a pas de wait fonction en C ou POSIX qui ne prend aucun argument.
2. Le deuxième problème est le scanf appelez, vous appelez avec *++, où *n prend la valeur de la mémoire pointée par n qui risque peut entraîner une panne. Supprimer l'astérisque.
3. Le troisième problème, c'est que vous traiter de la mémoire partagée à la fois comme un tableau d'entiers (avec n) et comme une chaîne de caractères. Vous ne pouvez pas vraiment faire les deux, choisir l'un ou l'autre.
4. Vous créez la mémoire partagée dans le processus parent, mais attendez que le processus de l'enfant à terminer avant de créer la mémoire.
5. Il y a une condition de concurrence entre le parent et l'enfant, puisque la part de la mémoire peut être créé après que l'enfant tente d'y accéder.

Modifier je suis venu avec ce lieu, qui semble fonctionner pour moi. J'ai ajouté des commentaires sur les choses que j'ai changé.

#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <stdio.h>
#include <sys/wait.h>  /* Needed for the wait function */
#include <unistd.h>    /* needed for the fork function */
#include <string.h>    /* needed for the strcat function */
#define SHMSIZE 27
int main() {
   int shmid;
   char *shm;

   if(fork() == 0) {
      shmid = shmget(2009, SHMSIZE, 0);
      shm = shmat(shmid, 0, 0);
      char *s = (char *) shm;
      *s = '
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <stdio.h>
#include <sys/wait.h>  /* Needed for the wait function */
#include <unistd.h>    /* needed for the fork function */
#include <string.h>    /* needed for the strcat function */
#define SHMSIZE 27
int main() {
int shmid;
char *shm;
if(fork() == 0) {
shmid = shmget(2009, SHMSIZE, 0);
shm = shmat(shmid, 0, 0);
char *s = (char *) shm;
*s = '\0';  /* Set first location to string terminator, for later append */
int i;
for(i=0; i<5; i++) {
int n;  /* Variable to get the number into */
printf("Enter number<%i>: ", i);
scanf("%d", &n);
sprintf(s, "%s%d", s, n);  /* Append number to string */
}
strcat(s, "\n");  /* Append newline */
printf ("Child wrote <%s>\n",shm);
shmdt(shm);
}
else {
/* Variable s removed, it wasn't used */
/* Removed first call to wait as it held up parent process */
shmid = shmget(2009, SHMSIZE, 0666 | IPC_CREAT);
shm = shmat(shmid, 0, 0);
wait(NULL);
printf ("Parent reads <%s>\n",shm) ;
shmdt(shm);
shmctl(shmid, IPC_RMID, NULL);
}
return 0;
}
';  /* Set first location to string terminator, for later append */
      int i;
      for(i=0; i<5; i++) {
         int n;  /* Variable to get the number into */
         printf("Enter number<%i>: ", i);
         scanf("%d", &n);
         sprintf(s, "%s%d", s, n);  /* Append number to string */
      }
      strcat(s, "\n");  /* Append newline */
      printf ("Child wrote <%s>\n",shm);
      shmdt(shm);
   }
   else {
      /* Variable s removed, it wasn't used */
      /* Removed first call to wait as it held up parent process */
      shmid = shmget(2009, SHMSIZE, 0666 | IPC_CREAT);
      shm = shmat(shmid, 0, 0);
      wait(NULL);
      printf ("Parent reads <%s>\n",shm) ;
      shmdt(shm);
      shmctl(shmid, IPC_RMID, NULL);
   }
   return 0;
}

Ne remarque que le point 5 dans la liste ci-dessus n'ont pas été résolus.

OriginalL'auteur Some programmer dude

Un problème, c'est que le processus de l'enfant est d'essayer d'utiliser get la mémoire partagée avant il a été créé par le parent. Le parent a un wait() appel avant la création de la mémoire partagée, donc il n'existe pas lorsque le client essaie de récupérer l'identifiant. Même si l'attente() l'appel est déplacé, il peut ne pas fonctionner car il y a une race condition. L'appel à shmget peut-être besoin de faire précéder le fork appel (ou d'utiliser certains de synchronisation pour s'assurer qu'il existe réellement avant de la récupérer dans le processus de l'enfant).

Et (comme d'autres l'ont déjà souligné), l'enfant est de tenter d'écrire des nombres entiers de la mémoire lors de la lecture (impression) essaie de la traiter comme une chaîne de caractères.

OriginalL'auteur Mark Wilkins