내 프로그램에서 segfault를 찾지 못했습니다.

Question

이 세분화 오류 11 오류의 출처를 찾으려고 시간을 보내고 있습니다. 여러분이 저를 도울 수 있다면 바라고 있습니다.

그래서 프로그램의 목적은 입력 문자열에서 리디렉션을 찾고 리다이렉션 방향 인 fileName과 argsWithoutFile을 제공하는 redirCode 구조체를 반환하는 것입니다.

예 입력 :

"ls -a > test.txt" 

가와 REDIR 코드를 반환

argsWithoutFile = "ls -a" 
fileName = "test.txt" 
code = 2      //stdout 

나는 독방 감금 오류가 하위 문자열을하려고에서 오는 확신 해요. 왜냐하면 내가 마지막 부분 문자열을 주석 처리 할 때 segfault를주지 않기 때문이다. 하위 문자열을 제외하고 모두 괜찮은 것 같습니다.

char *input = "ls -a > test.txt' 
char *desiredSubString = "ls -a " 

다음

---

는 전체 코드입니다 : : 기본적으로 문자열에 대한 I는이처럼되고 싶어 일부 터미널 로그는 여기에있다

#include <stdio.h> 
#include <string.h> 
#define BUFFER  1024 
struct redirCode findRedirects(char *input); 
struct redirCode { 
    /* For code: 
    * 0 = none 
    * 1 = stdin 
    * 2 = stdout 
    */ 
    int code; 
    char *argsWithoutFile; 
    char *fileName; 
}; 

int main(){ 
    const char *delims = "<>"; 
    struct redirCode temp; 
    char line[BUFFER]; 

    printf("Input: "); 
    fgets(line, 1024, stdin); 
    temp = findRedirects(line); 

    printf("temp:\n"); 
    printf("temp.code = %d\n", temp.code); 
    printf("temp.fileName = %s\n", temp.fileName); 
    printf("temp.argsWithoutFile = %s\n", temp.argsWithoutFile); 

} 

/* Looks for '>', '<' in a string. 
* Will destroy string *input 
* Returns a redirCode struct with: 
* 1. fileName - the name of file that 
* wants to be redirected/stdin 
* 2. code - the direction of redirect 
* 3. args - the arguments w/o filename 
* */ 
struct redirCode findRedirects(char *input) 
{ 
    const char *delims = "<>"; 
    struct redirCode redirToReturn; 

    //Do an initial search for the delimeters 
    //before strtok destroys it. O(n) time. 
    int redirectOperatorReached = 0; 
    int count = 0; 

    int i; 
    for (i = 0; input[i] != 0; i++){ 
     if (input[i] == '<'){ 
      redirToReturn.code = 1; 
      redirectOperatorReached = 1; 
     } 
     else if (input[i] == '>'){ 
      redirToReturn.code = 2; 
      redirectOperatorReached = 1; 
     } 
     else { 
      redirToReturn.code = 0; 
     } 
     if (redirectOperatorReached != 1){ 
      count++; 
     } 
    } 
    printf("sizeof(input) = %lu\n", sizeof(input)); 
    printf("count = %d\n", count); 
    strncpy(redirToReturn.argsWithoutFile, input, count); 
    printf("input = %s\n", input); 
    redirToReturn.argsWithoutFile[count] = '\0'; 
    printf("argsW/oFile = %s\n", redirToReturn.argsWithoutFile); 

    return redirToReturn; 
} 

그리고

MacBook-Air:practice keithy$ cc strtokOnlyOnce.c 
MacBook-Air:practice keithy$ ./a.out 
Input: hi 
sizeof(input) = 8 
count = 3 
Segmentation fault: 11 
MacBook-Air:practice keithy$ cc strtokOnlyOnce.c 
MacBook-Air:practice keithy$ ./a.out 
Input: ls -a > test.txt 
sizeof(input) = 8 
count = 6 
Segmentation fault: 11 
MacBook-Air:practice keithy$ 

---

편집 : 작동하도록했습니다! 내가해야 할 일은 redirCode의 문자열을 malloc하는 것이 었습니다. 여기에 작업 코드 :

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 
#define BUFFER  1024 
struct redirCode findRedirects(char *input); 
struct redirCode { 
    /* For code: 
    * 0 = none 
    * 1 = stdin 
    * 2 = stdout 
    */ 
    int code; 
    char *argsWithoutFile; 
    char *fileName; 
}; 

int main(){ 
    const char *delims = "<>"; 
    struct redirCode temp; 
    char line[BUFFER]; 

    printf("Input: "); 
    fgets(line, 1024, stdin); 
    temp = findRedirects(line); 

    printf("temp.code = %d\n", temp.code); 
    printf("temp.fileName = %s\n", temp.fileName); 
    printf("temp.argsWithoutFile = %s\n", temp.argsWithoutFile); 

} 

/* Looks for '>', '<' in a string. 
* Will destroy string *input 
* Returns a redirCode struct with: 
* 1. fileName - the name of file that 
* wants to be redirected/stdin 
* 2. code - the direction of redirect 
* 3. args - the arguments w/o filename 
* */ 
struct redirCode findRedirects(char *input) 
{ 
    const char *delims = "<>"; 
    struct redirCode *redirToReturn = malloc(sizeof(struct redirCode)); 

    //Do an initial search for the delimeters 
    //before strtok destroys it. O(n) time. 
    int redirectOperatorReached = 0; 
    int count = 0; 

    int i; 
    for (i = 0; input[i] != 0; i++){ 
     if (input[i] == '<'){ 
      redirToReturn->code = 1; 
      redirectOperatorReached = 1; 
      input[i] = ' '; 
     } 
     else if (input[i] == '>'){ 
      redirToReturn->code = 2; 
      redirectOperatorReached = 1; 
      input[i] = ' '; 
     } 
     if (redirectOperatorReached != 1){ 
      count++; 
     } 
    } 
    int lengthOfInput = strlen(input); 

    int sizeOfMalloc = (lengthOfInput+1)*sizeof(char); 
    redirToReturn->argsWithoutFile = (char *) malloc(sizeOfMalloc); 
    redirToReturn->fileName = (char *) malloc(sizeOfMalloc); 

    strncpy(redirToReturn->argsWithoutFile, input, count); 
    redirToReturn->argsWithoutFile[count] = '\0'; 

    strncpy(redirToReturn->fileName, input + count, lengthOfInput - count); 

    return *redirToReturn; 
} 

/*OUTPUT 
*./a.out 
*Input: ls -a > test.txt 
*temp.code = 2 
*temp.fileName = test.txt 
*temp.argsWithoutFile = ls -a 
*/ 

Answer 1

struct redirCode { 
     /* For code: 
     * 0 = none 
     * 1 = stdin 
     * 2 = stdout 
     */ 
     int code; 
     char *argsWithoutFile; 
     char *fileName; 
    }; 

당신은 그것을 사용하기 전에 char *argsWithoutFile; 및 char *fileName;라는 이름의 구조체 멤버에 대한 메모리를 할당해야합니다. 이

struct redirCode *redirToReturn = malloc(sizeof(struct redirCode)); 
redirToReturn->argsWithoutFile = malloc(1024); 
redirToReturn->fileName = malloc(1024); 

Answer 2

1> 당신은 초기화되지 strncpy 2> 파일 이름을하기 전에 new/malloc 사용 argsWithoutFile 에 메모리를 할당 할 필요가 당신은 printf와에 인쇄되어 있습니다. 이것은 정의되지 않은 동작입니다.