Iは、次のヘッダ（stack.h）に従ってCスタックを実装しようとしている

Question

NULLにポインタを設定することにより、Cスタックを初期化：

#ifndef STACK_H 
#define STACK_H 

/* An element from which stack is consisting */ 
typedef struct stack_node_ss { 
    struct stack_node_ss *next; /* pointer to next element in stack */ 
    void *value;     /* value of this element */ 
} stack_node_s; 

/* typedef so that stack user doesn't have to worry about the actual type of 
* parameter stack when using this stack implementation. 
*/ 
typedef stack_node_s* stack_s; 

/* Initializes a stack pointed by parameter stack. User calls this after he 
* has created a stack_t variable but before he uses the stack. 
*/ 
void stack_init(stack_s *stack); 

/* Pushes item to a stack pointed by parameter stack. Returns 0 if succesful, 
* -1 otherwise. 
*/ 
int stack_push(void *p, stack_s *stack); 

/* Pops item from a stack pointed by parameter stack. Returns pointer to 
* element removed from stack if succesful, null if there is an error or 
* the stack is empty. 
*/ 
void *stack_pop(stack_s *stack); 

#endif 

を但し、Cと新しいされ、私はstack_init機能で立ち往生、私はstack.cで書かれています：

#include <stdlib.h> 
#include <stdio.h> 
#include "stack.h" 

void stack_init(stack_s *stack) { 
    (*stack)->value = NULL; 
    (*stack)->next = NULL; 
} 

メインプログラムはで始まる：

int *tmp; 
    stack_s stack; 
    stack_init(&stack); 

そして、これは私のプログラムがクラッシュwith：

Program received signal EXC_BAD_ACCESS, Could not access memory. 
Reason: KERN_INVALID_ADDRESS at address: 0x0000000000000008 
0x0000000100000abf in stack_init (stack=0x7fff5fbffb30) at stack.c:6 
6  (*stack)->value = NULL; 

あなたは正しい方向に私をヒントできますか？どうもありがとう。

Answer 1

あなたは**stack自身のためにメモリを割り当てる必要があります：

*stack = malloc(sizeof(**stack)); 

しかし、ポインタ型をtypedefをしないでください。それは本当に混乱して読みにくいです。より良い、このように、値によってポインタを渡し、ポインタを格納するために、呼び出し元にそれを残すために：

typedef struct stack_node_t 
{ 
    struct stack_node_t * next; 
    /* ... */ 
} stack_node; 

stack_node * create_stack() 
{ 
    stack_node * res = calloc(1, sizeof(stack_node)); 
    return res; 
} 

void destroy_stack(stack_node * s) 
{ 
    if (!next) return; 

    stack_node * next = s->next; 
    free(s); 
    destroy_stack(next); 
} 

// etc. 

次に、あなただけ言うことができる：

stack_node * s = create_stack(); 

// use s 

destroy_stack(s); 
s = NULL; // some people like this 

Answer 2

あなたは未定義の原因となる、初期化されていないポインタを参照解除されています動作。

この機能は、新しいスタックを作成しているので、あなたがスタックのために、いくつかの動的メモリを割り当て、その新しく割り当てられたメモリを指すようにポインタを設定する必要があります。

void stack_init(stack_s *stack) { 
    *stack = malloc(sizeof(**stack)); // create memory for the stack 

    (*stack)->value = NULL; 
    (*stack)->next = NULL; 
} 

stack_s stack; 
stack_init(&stack); 

次にあなたが呼ばれる機能を持っている必要があります動的メモリをfreeとNULLへのポインタを設定しますstack_destroy：

​​

Answer 3

あなたはNULLにスタックを初期化する必要があります - それにNULL値をプッシュするではない：

void stack_init(stack_s *stack) { 
    *stack=NULL; 
}