Копирование подстрок

Question

Как всегда программирую под Android NDK.

Задача

Есть строка размером 32 байта. Нужно скопировать две подстроки по 8 байт с определенной позиции, а затем получить код каждого символа.

Написал код:

const char* str = "0a88d2e044ed675309eb79c6aa988ce2";

char* test1(){
    char k[17];

    strncpy(k, &str[3], 8);
    strncpy(k + 8, &str[8], 8);

    k[16] = 0;
    return k;
}

char* test2(){
    char* k = (char*) malloc(17 * sizeof(char));
    memset(k, 0, 17);

    memcpy(k, str + 3, 8);
    memcpy(k + 8, str + 8, 8);

    return k;
}

При вызове функций получаю одинаковый результат: 8d2e044e44ed6753.

Перебираю результат и получаю код символа:

c = test1 = test2
8 = 0     =  56
d = 17    =  100
2 = 0     =  50
e = 0     =  101
0 = 0     =  48
4 = 0     =  52
4 = 0     =  52
e = 0     =  101
4 = 0     =  52
4 = 17    =  52
e = 0     =  101
d = 0     =  100
6 = 0     =  54
7 = 17    =  55
5 = 0     =  53
3 = 0     =  51

test1 использует strncpy на экране — одно, а код — другой (видимо, кодировка своя какая-то).

Решил использовать test2, появились вопросы:

1. Обнуляю выделенную память в начале через memset. Как ее обнулить после расширения памяти realloc-ом? Или, например, выделил памяти 30 байт, использовал 10, как поставить 0 в конце строки, чтобы не получить весь хлам до первого нуля?

2. Функция test2 вернула результат, я что-то сделал. Как освободить выделенную внутри test2 память? Если освободить память так (см. ниже), то при следующем вызове функция вернет всякий хлам.

   char* test2() {
       char* k = (char*) malloc(17 * sizeof(char));
       memset(k, 0, 17);
       memcpy(k, str + 3, 8);
       memcpy(k + 8, str + 8, 8);
       const char* nk = k;
       free(k);
       return nk;
   }

3. Как получить размер выделенной памяти?

   char* k = (char*) malloc(17 * sizeof(char));

   Результат sizeof(k) говорит о том, что выделено 8 байт. Но, если я правильно понимаю, в результате должно быть 17 байт.

Answer 1

1 Нельзя узнать размер объекта по указателю. Ваша ошибка в том, что вы не храните этот размер самостоятельно. Используйте, например, такую логику:

typedef struct MyData {
    int   size;
    char *data;
} MyData;

MyData *CreateData(int size){
    MyData *tmp = (MyData*)malloc(sizeof(MyData));

    if(tmp==NULL) // Что то пошло не так...
        exit(-1);

    tmp->data = (char*)malloc(size);

    if(tmp->data==NULL) // Что то пошло не так...
        exit(-1);

    tmp->size = size;
    return tmp;
}

void ReallocData(MyData *data, int size){
    char *tmp = (char*)realloc(data->data, size);

    if(tmp==NULL) // Что то пошло не так...
        exit(-1);

    data->data = tmp;
    data->size = size;
}

void DeleteData(MyData *data){
    free(data->data);
    free(data);
}

Соответственно, никаких проблем не будет с определением текущего размера. По поводу memset, аналогично, зная размеры выделенной памяти всё становится понятным.

2 Вы неверно сделали логику функции. вот пример основанный на логики первого пункта ответа:

MyData *test2() {
    MyData *tmp = CreateData(17);
    memset(tmp->data, 0, 17);
    memcpy(tmp->data, str + 3, 8);
    memcpy(tmp->data + 8, str + 8, 8);
    return tmp;
}

MyData *data = test2(); // получаем данные
... // работаем с данными

DeleteData(data); // чистим за собой

3 Вы удаляете данные, которые хотите вернуть. Это неверное решение. Данные нужно удалять когда они уже не нужны.

4 Ваша фраза ошибочна:

sizeof(k) говорит выделили 8 байт

sizeof от указателя возвращает размер указателя. А объект у вас char, его sizeof всегда 1 байт. Первый пункт объясняет как "измерять" данные, которые вы выделили - Никак. Только запоминать самому.