0

だから私はCでプログラムされたシェルをデバッグしようとしています

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define MAX_NUM_ARGS 256
#define SIZE 256

//void orders(char *command[SIZE]);

int main() {

    char buffer[SIZE]= "";
    //char input_args[MAX_NUM_ARGS];
    char **input_args = NULL;
    int i = 0;// counting variable
    int j = 0;// second counting variable (thank you Nathan)
    int next_counter = 0;

    printf("Welcome to my shell.\n");
  while(1){

    // sees to it that the buffer is clean (thanks erik =) )
    memset(buffer, '\0', sizeof(buffer));
    i = 0;
    j = 0; //ensure that the counting variables are reset


    //initialize array of strings
    //first free any prevously allocated memory
    if (input_args != NULL)
    {   //memory has been allocated free it
        for (i = 0; i <MAX_NUM_ARGS; i++)
        {
            free(input_args[i]);
        }
    }   
    //free array of strings
    free(input_args);

    //new allocate memory
    input_args = (char**) malloc(sizeof(char*) * MAX_NUM_ARGS);
    //check return value for error
    if (input_args == NULL)
    {
        printf("We are out of memory. =( Can't run. Sorry!\n");

        return -1; //Thank you Erik for this idea!
    }
    //allocate memory for each string
    for (i = 0; i <MAX_NUM_ARGS; i++)
    { 
        input_args[i]= (char*)malloc(sizeof(char) * MAX_NUM_ARGS);
        if(input_args[i] == NULL)
            {//error
            printf("Error, the input is empty.");
            return -1;
            }//end of if statement
    }//end of for loop


    printf("~$: "); //prompts the user for input
    fgets(buffer, sizeof(buffer), stdin);
    //if the user types in exit, quit
    if (strcmp(buffer, "exit\n") == 0){ 
        exit(0);
    } //end of if statement
    //if user types in clear, wipe the screen and repeat the loop
    else if(strcmp(buffer, "clear\n")==0){

        system("clear");    
        continue;   

    }//end of else if
    //should the user punch in nothing, repeat the loop
    else if (strcmp(buffer, "\n") == 0) {
        continue;
    }//end of else if




    for (i = 0; i < SIZE; i++) {

        if(buffer[i] != '\n' && buffer[i] != ' ' && buffer[i] != '\t'){

            input_args[j][i] = buffer[i];
        }   //end of if statement
        else{
            input_args[j][i] = '\0';
            j++;

        }//end of else statment

    }//end of for loop

    input_args[1] = NULL;

    //block down here handles the command arugments
    int retval = 0; //return value
    int pid = 0;
    int childValue = 0;
    pid = fork();

    if (pid != 0){
    //  printf("I'm the parent, waiting on the child.\n");//debug
        pid = waitpid(-1, &childValue,0);
    //  printf("Child %d returned a value of %x in hex.\n", pid, childValue);

    }//end of if statement
    else{
    //  printf("I am the first child.\n");
        retval = execvp(input_args[0], input_args);
        //exit(2);
        if (retval != -1){
            //print error!
            printf("Invalid command!\n");
            exit(2);
        }
    }//end of else block

   } //end of while loop
    return 0;

}//end of main function

これで、このシェルに 'ls' や 'pwd' などの 1 語のコマンドを実行させるか、vi に入って新しいファイルを開くことができます。しかし、複数単語の引数はうまく機能していないようです。

コードの基本的なロジックに問題があります。つまり、一番下のコード ブロックを見ると、両方の引数を受け取るようにコーディングされているように見えますが、現在は最初の引数だけが解析されています。私は正確にどのような論理エラーを起こしていますか? 私はこれについて学ぶことに興味があります。

4

1 に答える 1

1

たぶんそれが唯一の問題です:あなたが持っている引数を解析するループの後

input_args[1] = NULL;

そのため、以前に行ったことは何であれ、これ以上引数はありません (input_args[0]はプログラム名です)。きっとあるはず

input_args[j] = NULL;

編集

これでシェルが機能する可能性がありますが、最初にすべてにメモリを割り当てると、メモリ リークが発生しますinput_args[i](0 <= i < MAX_NUM_ARGS)。input_args[j] = NULLそのため、そのメモリを設定すると、二度と解放されません。あまりエレガントではありませんが、実用的な解決策は呼び出すことです

free( input_args[j] );

前に、実際に必要な引数にのみメモリを割り当てることをお勧めします。

于 2013-09-26T16:57:24.630 に答える