Linux Execve, ошибка сегментации (strcmp_sse42) - PullRequest
Новая
       60

Linux Execve, ошибка сегментации (strcmp_sse42)

0 голосов
/ 13 мая 2018 
#include <stdio.h>
#include <unistd.h>
#include <string.h>
#include <stdlib.h>

void main(){
   int x,y,status, i;
   int cnt = 0;
   int flag = 0;
   char buf[50];
   char str[50];
   char * argv[10];
   char * ptr;


   for(i=0; i<10; i++){
    printf("$");
    gets(buf);
    strcpy(str, buf);

    ptr = strtok(buf, " ");

    while(ptr != NULL){
      argv[cnt] = ptr;
      cnt++;
      ptr = strtok(NULL," ");
    }



    if(!strcmp(argv[cnt-1], "&")) {
      argv[cnt-1] = 0;
      flag = 1;
    }
    else {
        argv[cnt] = 0;
    }



    if(!strcmp(argv[cnt-1], "exit")) exit(0); 

    x=fork();

    if (x==0){
        sleep(1);
        printf("I am child to execute %s\n", str);
        y=execve(argv[0], argv, 0);

        if (y<0){
           perror("exec failed");
           exit(1);
        }

    }
    else {
      if(flag == 0) { 
          wait(&status); 
      }
    }


    flag = 0;
    cnt = 0;
   }
}

запустить этот код в Linux, затем segement falut (основной дамп)

также при использовании gdb,

================================================================

Программа получила сигнал SIGSEGV, Ошибка сегментации.0x0000003b6572fa96 в __strcmp_sse42 () из /lib64/libc.so.6

========================================================================

почему это не работает?

, если я набираю /bin / ls -al (что-либо без '&') хорошо работает

тип буфера / bin / ls -al & error

1 Ответ

0 голосов
/ 13 мая 2018

Если вы введете &, переменная argv[cnt-1] будет установлена ​​в NULL, поэтому в if(!strcmp(argv[cnt-1], "exit")) первый аргумент функции strcmp будет равен NULL, что приведет к падению приложения ...

Кроме того, ваш код не проверяет переполнение буфера, индекс выходит за границы, ... Этот код довольно «опасен»

Это означает, что:

  • Вы не должны использовать gets, см. https://stackoverflow.com/a/1694042/808101
  • Убедитесь, что argv (обратите внимание: это имя переменной, возможно, не очень хорошее, argv очень часто является аргументами самой программы) достаточно большой массив при вставке в него элемента
...