Execl返回坏地址

Question

我真的很想在这个版本中提供一些调试帮助。从早上4点开始我就一直在做这个(我应该在上午7小时11点内送来这个)

main.c中的所有内容都可以工作，但是当我创建一些子进程来使用execl运行Compute.c的编译文件时，它不会这样做，并且会发送一个“坏地址”错误。

我已经附上了3个带有main.c和compute.c的pastebin链接，以及一个包含下面提到的表的txt文件。

该程序应该从一个名为pinakes.txt的文件中读取两个带有整数的表，然后使用POSIX的共享内存API将这些表放置在共享内存中，并创建进程从这些表中计算一个“行*列”和，并将该和放在另一个表中。

sum += A[row][i] * B[i][column] = C[row][column]

在main.c下面的行之前，所有内容都应该正常工作(我多次调试它)。

ppid = getpid();

• main.c http://pastebin.com/iMCefaLZ
• compute.c http://pastebin.com/Ejp214Up
• pinakes.txt http://pastebin.com/h8yKXFvv

​

编译然后运行

./main pinakes.txt

main.c

188行代码

#include <stdio.h>
#include <sys/types.h>
#include <unistd.h>
#include <sys/wait.h>
#include <signal.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include <sys/shm.h>
#include <sys/ipc.h>
#include <errno.h>

int pinA_X = 0, pinA_Y = 0, pinB_X=0, pinB_Y=0;
int pinA[10][10], pinB[10][10], pinC[10][10];

main(int argc, char *argv[]){
    int pid, ppid;
    FILE *stream;
    // general variables
    int i, c, j, rc, converted, lines = 0;
    //flags
    int flagV=0, flagW=0, flagX=0, flagY=0, flagZ=0;

    //shared memory
    int dumpedArray[101];
    int size = sizeof(dumpedArray);
    int sid1 = shmget(IPC_PRIVATE, size, SHM_R | SHM_W);
    int sid2 = shmget(IPC_PRIVATE, size, SHM_R | SHM_W);
    int sid3 = shmget(IPC_PRIVATE, size, SHM_R | SHM_W);
    int* shared_A = (int*) shmat(sid1, NULL, 0);
    int* shared_B = (int*) shmat(sid2, NULL, 0);
    int* shared_C = (int*) shmat(sid3, NULL, 0);

    if(argc!=2){
        printf("wrong number of arguments\n");
        return -1;
    }else{
        stream = fopen(argv[1] , "r");
                while((c = getc(stream))!= EOF){
            if(flagZ == 0){
                if(flagX == 1){pinA_X = c - 48;flagX = 0;}
                if(c == 88){flagX = 1;}
                if(flagY == 1){pinA_Y = c - 48;flagY = 0;}
                if(c == 89){flagY = 1;}
                if(c == 90){flagZ = 1;}
            }else if(flagZ == 1){
                if(flagX == 1){pinB_X = c - 48;flagX = 0;}
                if(c == 88){flagX = 1;}
                if(flagY == 1){pinB_Y = c - 48;flagY = 0;}
                if(c == 89){flagY = 1;}
            }
        }

        fclose(stream);

        printf("pinA[%d][%d] * pinB[%d][%d] = C[%d][%d]\n\n", pinA_X, pinA_Y, pinB_X, pinB_Y, pinA_X, pinB_Y);

        // get A
        stream = fopen(argv[1] , "r");
        i=0;j=0;
        while((c = getc(stream))!= EOF){
            if(i <= pinA_X && j <= pinA_Y){
                if(flagW == 0){
                    if(c == 87){
                        flagW = 1;
                    }
                }else{
                    if(c > 47 && c < 58){
                        pinA[i][j] = c - 48;
                        j++;
                    }
                    if(c == 13){
                        j=0;
                        i++;
                    }
                }
            }
        }
        fclose(stream);

        // get B
        stream = fopen(argv[1] , "r");
        i=0;j=0;
        while((c = getc(stream))!= EOF){
            if(i <= pinB_X && j <= pinB_Y){
                if(flagV == 0){
                    if(c == 86){
                        flagV = 1;
                    }
                }else{
                    if(c > 47 && c < 58){
                        pinB[i][j] = c - 48;
                        j++;
                    }
                    if(c == 13){
                        j=0;
                        i++;
                    }
                }
            }
        }
        fclose(stream);

        // print A

        printf("A={\n");
        for(j=0; j<pinA_X;j++){
            for(i=0;i<pinA_Y;i++){
                printf(" %d", pinA[j][i]);
            }
            printf("\n");
        }
        printf("}\n\n");

        // print B

        printf("B={\n");
        for(j=0; j<pinB_X;j++){
            for(i=0;i<pinB_Y;i++){
                printf(" %d", pinB[j][i]);
            }
            printf("\n");
        }
        printf("}\n");

        // Save pinA to shared Memory
        converted = 0;

        for(i=0;i<10;i++){
            for(j=0;j<10;j++){
                converted = (i * 10) + j;
                shared_A[converted] = pinA[i][j];
            }
        }


        // Save pinA to shared Memory
        converted = 0;

        for(i=0;i<10;i++){
            for(j=0;j<10;j++){
                converted = (i * 10) + j;
                shared_B[converted] = pinB[i][j];
            }
        }

        // Push size of arrays in shared memory
        shared_A[100] = pinA_X;
        shared_A[101] = pinA_Y;
        shared_B[100] = pinB_X;
        shared_B[101] = pinB_Y;

        ppid = getpid();

        for(i=0; i<pinA_X; i++){
            for(j=0; j<pinB_Y; j++){
                if(ppid == getpid()){
                    pid = fork();
                    if(pid==0){
                        if(execl("./compute", "compute", i, j, sid1, sid2, sid3, NULL) == -1){
                            printf("error exec\n");
                            printf("Error opening file: %s\n", strerror(errno));
                        };
                    }else if(pid<0){
                        printf("\nDen egine h fork!\n");
                    }else{
                        wait(0);
                    }
                }
            }
        }

        //print C
        converted = 0;

        printf("C={\n");
        for(i=0;i<10;i++){
            for(j=0;j<10;j++){
                converted = (i * 10) + j;
                pinC[i][j] = shared_C[converted];
                printf(" %d", pinC[i][j]);
            }
            printf("\n");
        }
        printf("}\n");
    }
}

compute.c和pintakes.txt都与回答这个问题没有直接关系。

Answer 1

出现错误地址问题是因为您运行：

for(i=0; i<pinA_X; i++){
    for(j=0; j<pinB_Y; j++){
        if(ppid == getpid()){
            pid = fork();
            if(pid==0){
                if(execl("./compute", "compute", i, j, sid1, sid2, sid3, NULL) == -1){

execl()的参数必须是字符串；i和j显然不是字符串( sid1、sid2和sid3是三块共享内存的标识符)。

将这些值转换为字符串，然后再试一次。

您的程序使用IPC_PRIVATE创建共享内存，因此compute.c中的代码(通过execl()执行)将很难工作。您可以这样传输共享内存I，但我不确定。

我想我会使用一个共享内存段。

您的读取代码看起来也会将相同的数据读入这两个数组--但我可能误读了它。

最后，您的PasteBin示例在23小时内过期。这限制了你的问题的有用性。你真的应该把数据转移到问题中--我建议，不要将制表符和制表符设置为4而不是8 (或者使用更多的函数来防止如此深的缩进。)

Answer 2

您要将ints传递给execl，这些都应该是以0结尾的字符串.另外，必须将最终的NULL转换为Char*。