Question

我收到错误细分错误（核心转储）

我已经在函数threadx

中缩小了这些行

    while (colatz_nums[j] != 1)
         {j++;
            if ((m % 2)==0)
            { colatz_nums[j]  = m/2;}

            else
            {colatz_nums[j]  = 3 * m +1;}
        }

如果我删除这些行，我不会收到错误。我在循环中添加了一个测试并且它有效所以它必须是这些方面的东西。请指出错误

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>  // pid_t
#include <unistd.h>
#include <sys/ipc.h>
#include <sys/shm.h>
#include <sys/mman.h>
#include <sys/fcntl.h>
#include <sys/stat.h>

#include <pthread.h>

#define N 2

void *thread (void *vargp);
void *threadx(void *vargp);

char **ptr;

int fib_nums[25] ;
int colatz_nums[25];
int last_collatz = 0;

int main()
{

    int i;
    pthread_t tid[2];
    char *msgs[N] = {
        "Hello from foo",
        "Hello from bar"
    };

    printf("Parent thread started with PID= %d and parent PID %d\n", getpid(), getppid());

    ptr = msgs;

    pthread_create(&tid[0], NULL, thread, (void *)1);
    printf(" 1st thread started with PID= %d and parent PID %d\n", getpid(), getppid());


    pthread_create(&tid[1], NULL, threadx, (void *)2 );
    printf("2nd  thread started with PID= %d and parent PID %d\n", getpid(),      getppid());

    pthread_join(tid[1], NULL);
    pthread_join(tid[0], NULL);
}

void *thread(void *vargp)
{
    int myid = (int)vargp;
    static int cnt = 0;
    printf(" thread ");

    int i=cnt;
    for (;i <10 ;i=i+1)
    {
        printf("[%d]  %d\n",myid, i);
        sleep(cnt);
    }

    return NULL;
}

void *threadx(void *vargp )
{
    int myid = (int)vargp;
    static int cnt = 0;
    printf(" threadx \n" );
    int j = 0;
    int m = 8;
    colatz_nums[0] = 8;

    while (colatz_nums[j] != 1)
    {
        j++;
        if ((m % 2)==0)
        {
            colatz_nums[j]  = m/2;
        }

        else
        {
            colatz_nums[j]  = 3 * m +1;
        }
    }
    last_collatz = j;

    for (j=0; j <= last_collatz; j++)
        printf ( " j %d",colatz_nums[j]);

    printf ( "\n");
    return NULL;
}

Answer 1

您永远不会检查colatz_nums的界限。您正在使用j访问它并将其递增而不将其限制为24。

首先执行

colatz_nums[0] = 8

将数组的第一个值设置为8.然后你将它与1进行比较，然后循环直到你在数组中找到1。

你的循环中的问题是你首先递增j然后设置位于索引j的值（这是你将在循环的下一轮中对1测试的下一个值）到a值永远不会是1（4或25，但在你的例子中总是4）。

然后，您将永远循环，直到崩溃发生（超出限制访问）。

Answer 2

m 永远不会更改，因此colatz_nums[j] 连续设置为4（因为m为8 ，一个偶数），一直到你离开阵列末尾的点。

你可以通过简单地将这一行作为while循环中的最后一行来解决这个问题：

m = colatz_nums[j];

或将其重写为类似于避免未定义行为的更安全的变体：

while (colatz_nums[j] != 1) {
    j++;
    if ((m % 2)==0)
        m  = m / 2;
    else
        m  = 3 * m + 1;

    if (j == sizeof(colatz_nums) / sizeof(colatz_nums[0])) {
        fprintf (stderr, "Buffer overflow\n");
        exit (1);  // or some other method of stopping
    }
    colatz_nums[j]  = m;
}

为什么发生分段错误错误

2 个答案: