Странное поведение sscanf со строкой

Question

У меня проблема с sscanf. Чтобы проверить это, я сделал простой файл, поэтому, когда я компилирую это:

#include <stdio.h> 
main(){ 
    char *a; 
    /* */ 
    char *s = "GET /something HTTP/1.1\r\n"; 
    printf("sscanf: %d\n", sscanf(s, "GET %s HTTP", a)); 
    printf("a: %s\n", a); 
    /* */ 
    printf("sscan: %d\n", sscanf("GET /more HTTP/1.1\r\n", "GET %s HTTP", a)); 
    printf("a: %s\n", a); 
} 

Я получаю правильный вывод:

sscanf: 1 
a: /something 
sscan: 1 
a: /more 

Но когда я комментирую линии между пустыми комментариями поёт, я получаю:

sscan: 0 
a: (null) 

Вопрос 1: Как это может быть так?

И еще немного: если я пишу char *a = NULL, я получаю:

sscanf: 0 
a: (null) 
sscan: 0 
a: (null) 

Вопрос 2: Почему?

Answer 1

Вы записываете данные в нераспределенное пространство. char *a; - это только указатель на случайную память. После того, как вы попытаетесь поместить данные в свою «строку», вы вызываете неопределенное поведение, и все может произойти.

Answer 2

Вы передаёте неинициализированный указатель на sscanf. измените char *a; на char a[100] и он должен работать правильно.

Answer 3

Вы копируете строку неинициализированному указателю (a). Вам необходимо выделить хранилище для него (с malloc) или объявить массив.

Реализации иногда могут работать правильно, если вы не выделяете какое-либо хранилище. Когда вы назначаете NULL a, sscanf ничего не хранит, поэтому вы пытаетесь напечатать «строку» с помощью указателя NULL (который printf печатает как (null)). В некоторых системах ваша программа просто сработает.

Существует расширение для компилятора/библиотеки, которое позволяет автоматически выделять строки, но, очевидно, это не переносимо, и не все компиляторы поддерживают его.

char *a; 
scanf("%ms", &a;) // allocates storage for a 

Вы можете передать неиницализированный указатель на функцию, но обратите внимание на &.

Answer 4

Ну, единственно правильным в моем случае, как мне кажется, было бы это:

#include <string.h> 
#include <stdio.h> 
#include <stdlib.h> 

main(){ 
    char *s = strdup("GET /something HTTP/1.1\r\n"); 
    char *a, *e, *ans; 
    char *end = s + strlen(s); 
    a = strstr(s, "GET"); 
    if(!a) return -1; 
    a += 4; 
    if(a >= end) return -1; 
    e = strstr(a, "HTTP/1.1"); 
    if(!e) return -1; 
    *(--e) = 0; 
    ans = strdup(a); 
    printf("ans: %s\n", ans); 
} 

В общем случае строка s может иметь очень большой размер, так что было бы лучше не выделять много памяти, но выделите право так, как мне нужно.

Корпус с "% as" не работает с моим gcc (4.7.2).