メモリ解放時にHEAP CORRUPTION DETECTEDエラーが発生する。

今日、コードをノックしたら、HEAP CORRUPTION DETECTEDエラーが出た。ポインタでメモリを確保し、最後にメモリ空間を解放するのは当たり前のことなのだが、エラーが出てしまったのだ。主に2つのメモリモデルから3つ目のメモリモデルにデータをコピーして、出力を以下のようにソートするコードを載せておきます。

#define _CRT_SECURE_NO_WARNINGS
#include"stdio.h"
#include"stdlib.h"
#include"string.h"

int getzip(char** p1, int num1, char (*p2)[30], int num2, char*** p3, int *num) {
    int ret = 0;
    char** tmp = NULL;
    int num3 = 0;
    if (p1 == NULL || p2 == NULL || p3 == NULL) {
        ret = 1;
        printf("func getzip passed in empty address: %d\n", ret);
        return ret;
    }
    num3 = num1 + num2;
    tmp = (char**)malloc(sizeof(char*) * num3);
    if (tmp == NULL) {
        ret = 1;
        printf("func getzip memory allocation failed: %d\n", ret);
        return ret;
    }
    for (int i = 0; i < num1; i++) {
        int len = strlen(p1[i]) + 1;
        tmp[i] = (char*)malloc(len);
        if (tmp[i] == NULL) {
            ret = 2;
            printf("func getzip memory allocation failed: %d\n", ret);
            goto END;
        }
        strcpy(tmp[i], p1[i]);
        //tmp[i][len] = '\0'; //error is here
    }

    for (int i = 0; i < num2; i++) {
        int len = strlen(p2[i]) + 1;
        tmp[num1 + i] = (char*)malloc(len);
        if (tmp[i] == NULL) {
            ret = 2;
            printf("func getzip memory allocation failed: %d\n", ret);
            goto END;
        }
        strcpy(tmp[num1 + i], p2[i]);
        //tmp[num1 + i][len] = '\0'; //error is here
    }

END:
    if (ret ! = 0) {
        if (tmp ! = NULL) {
            for (int i = 0; i < num3; i++) {
                if (tmp[i] ! = NULL)
                    free(tmp[i]);
            }
            free(tmp);
        }
    }
    else {
        *p3 = tmp;
        *num = num3;
    }
    return 0;
}

int print_str(char** p3, int num3) {
    int ret = 0;
    if (p3 == NULL) {
        ret = 1;
        printf("func print_str passed in null address: %d\n", ret);
        return ret;
    }

    for (int i = 0; i < num3; i++) {
        if(p3[i] ! = NULL)
            printf("%s\n", p3[i]);
    }

    return 0;
}

int sort_str(char** p3, int num3) {
    int ret = 0;
    char* tmp = NULL;

    if (p3 == NULL) {
        ret = 1;
        printf("func print_str passed in null address: %d\n", ret);
        return ret;
    }

    for (int i = 0; i < num3; i++) {
        for (int j = i + 1; j < num3; j++) {
            if (strcmp(p3[i], p3[j]) > 0) {
                tmp = p3[i];
                p3[i] = p3[j];
                p3[j] = tmp;
            }
        }
    }

    return 0;
}

void freemem(char*** p3, int num3) {
    int ret = 0;
    char** tmp = NULL;
    if (p3 == NULL || *p3 == NULL)
        return;

    tmp = *p3;
    for (int i = 0; i < num3; i++) {
        if (tmp[i] ! = NULL)
            free(tmp[i]);
    }
    free(tmp);
    *p3 = NULL;
    return ;
}

int main(){
    int ret = 0;
    int num1 = 3, num2 = 4;
    int num3 = 0;
    char *p1[] = {
"222222", "3333333333", "1111111111111"};
    char p2[4][30] = {
"nnnnnnn", "ooooooooooo", "dddddddddd", "bbbbbbbbbb"};
    char** p3 = NULL;

    getzip(p1, num1, p2, num2, &p3, &num3);

    print_str(p3, num3);

    sort_str(p3, num3);

    print_str(p3, num3);

    freemem(&p3, num3);

    system("pause");
    return 0;
}

strcpyが'Ⅻ'をまとめてコピーすることを最初忘れていました。ちなみにstrcpyの実装のソースコードはこちらに掲載されています。

char *strcpy(char *strDest, const char *strSrc);
{
assert((strDest!=NULL) && (strSrc ! =NULL)); 
char *address = strDest; 
while( (*strDest++ = * strSrc++) ! = '\0' ) 
NULL ;
return address ; 
}

そして、独自の文章を追加。

tmp[i][len] = '\0'; 

もともと、メモリ空間だけlen長を開き、私はlen + 1長を使用すると考えられている、アウトオブバウンズに似ているので、エラーのリリースが報告されます、私はそれが空間のリリースでなければならないと感じて、元の使用は、このポインタが指す空間に属していないチェック、それはエラーでリリースされます。に変更された場合

tmp[i][len - 1] = '\0'; 

この方法で解放しても問題はないので、解放の問題はかなりメモリ空間の確保や使い方に問題があるような気がします。