Question

我编写了以下程序，它模拟了信号量的工作。有三个功能：锁定，解锁，锁定路径。

lock =打开文件;检查文件是否已存在，如果存在，则将当前进程置于休眠状态。如果该文件不存在，则创建该文件并返回TRUE。

解锁 =删除文件

lockpath =返回与可能创建的文件对应的路径名。

这是源代码：

    #include <unistd.h>

//exit();
#include <stdlib.h>

//errno
#include <errno.h>

//creat(..)
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>

//strcat, strcpy
#include <string.h>

//For err_sys
#include <stdio.h>

#define LOCKDIR "/tmp/"
#define MAXTRY 3
#define WAITTIME 5

enum BOOLEAN{TRUE, FALSE};

void err_sys(const char* x) {
  perror(x);
  exit(1);
}

static char* lockpath(char* name) {
  static char path[20];
  strcpy(path, LOCKDIR);
  return (strcat(path, name));
}

int lock(char* name) {
  char *path;
  int fd, incerc;
  extern int errno;
  path = lockpath(name);
  int try = 0;

  while ((fd = open(path, O_WRONLY | O_CREAT | O_EXCL, 0666)) < 0 
          && errno == EEXIST) {
    if (++try >= MAXTRY)
        return FALSE;
    sleep(WAITTIME);  
  }

  if (fd < 0 || close(fd) < 0)
    err_sys("lock");

  return TRUE;
}

void unlock(char* name) {
  if (unlink(lockpath(name)) < 0)
    err_sys("unlock");
}


int main(void) {

  pid_t child_process;

  child_process = fork();

  char* sem_file_name = "test_semaf";

  if (child_process != 0)
  {
    printf("\nParent process ID: %d", getpid());
  }
  else 
  { 
    printf("\nChild process ID: %d", getpid());
  }

  if (lock(sem_file_name))
  {
      printf("\nProcess with ID: %d", getpid());
      printf("\nonly, has access to %s", strcat(LOCKDIR, sem_file_name)); //****
      unlock(sem_file_name);
  } else {
    printf("\nProcess with ID: %d", getpid());
    printf("\nwas unable to get access to %s", strcat(LOCKDIR, sem_file_name));
  }

  return 0;
}

程序停止的行标有：****

错误是：

编程接收信号SIGSEGV，分段故障。 __strcat_ssse3（）at ../sysdeps/x86_64/multiarch/strcat-ssse3.S:571 571 ../sysdeps/x86_64/multiarch/strcat-ssse3.S：没有这样的文件或目录。

问题是我得到了Segmentation Fault，并且找不到问题所在。对我来说，一切都很好。一个进程应该创建文件X.然后，如果另一个进程尝试创建它自己的文件X，则不允许;这个过程进入了睡眠状态。允许第二个进程进行MAXTRY尝试。如果在MAXTRY尝试后它没有成功，则lock（）函数返回FALSE。最后，当一个已成功创建自己的X文件的进程现在不需要它时，文件X将被删除。

请告诉您，您认为该计划的问题是什么？提前谢谢。

编辑：这是该页面的链接，解释了为什么lockpath（）函数不正确。

Is returning a pointer to a static local variable safe?

Answer 1

这是导致崩溃的原因：

Dest_ZIP    Ground
004-005     003
009         005
068-089     002
100-118     001

在这里，您尝试附加到文字字符串常量。

您也应该在这里使用strcat(LOCKDIR, sem_file_name)功能。

Answer 2

问题似乎在于你对strcat()功能的误解。该函数将第二个参数中的字符串附加到第一个参数 - 中的字符串，但您需要确保有足够的空间来存储数据。阅读man page。

这意味着

char * dest = "whatever";
strcat(dest, anything_else);

总是错的。你想要的是

char dest[SIZE] = "whatever";
strcat(dest, anything_else);

其中SIZE足够大，缓冲区能够包含整个连接字符串。

此外，您的lockpath()功能已损坏。请参阅this answer了解原因。您需要在lockpath()函数之外创建 dest 缓冲区，并将其作为参数传递给它。

信号量模拟程序：分段错误错误

2 个答案: