Valgrindのの不正な書き込み

Question

クラス：

class Tile { 
public: 
    long long size, joint; 
}; 

私の主が持っている：ここ

int main() { 
    // 
    double minW, minL; 
    unsigned tileCap = 10, tileCount = 0; 
    bool wasTransformed = false; 
    auto *tiles = (Tile*)calloc(tileCap, sizeof(Tile)); 

    GetInput(&minW, &minL, &tiles, &tileCount, &wasTransformed); 

    //etc. 
} 

問題の関数はGetInput()です：

void GetInput(double *w, double *l, Tile **tiles, unsigned *tileCount, bool *needTransform) { 
    // 
    printf("Min dimensions:\n"); 

    if (scanf("%lf %lf", w, l) != 2) 
    BadInput(); 
    if (!CorrectSize(*w) || *w == 0) 
    BadInput(); 
    if (!CorrectSize(*l) || *w == 0) 
    BadInput(); 

    unsigned tileCap = 10; 
    *tiles = (Tile*)calloc(tileCap, sizeof(Tile)); 

    printf("Tiles:\n"); 
    double tileSize, tileJoint; 
    int argc; 

    do { 
    argc = scanf("%lf %lf", &tileSize, &tileJoint); 
    if(argc == EOF) { 
     break; 
    } 
    if (tileSize == 0 || !CorrectSize(tileSize) || !CorrectSize(tileJoint) || argc != 2) 
     BadInput(); 

    if(! *needTransform) { 
     *needTransform = HasFloatingPoint(tileSize) || HasFloatingPoint(tileJoint); 
     if(*needTransform) 
     TransformPrevious(*tiles, *tileCount); 
    } 
    if(*needTransform) { 
     //transform this 
     tileSize *= 10; 
     tileJoint *= 10; 
    } 

    (*tiles)[*tileCount].size = (long long)tileSize + (long long)tileJoint; 
    (*tiles)[*tileCount].joint = (long long)tileJoint; 

    *tileCount += 1; 
    if((*tileCount) == tileCap) { 
     DoubleArray(tiles, &tileCap); 
    } 
    } while(true); 
} 

そして、私のDoubleArray()：

void DoubleArray(Tile **array, unsigned *cap) { 
    // 
    auto *tmp = (Tile*)realloc(*array, 2 * sizeof(Tile)); 

    if(tmp) { 
    *array = tmp; 
    *cap *= 2; 
    (*cap)--; 
    } else { 
    printf("Error allocating memory.\n"); 
    } 
} 

プログラムを実行するとエラーは表示されず、結果は正しいようです。たとえば、次のように12.3 0プリントInvalid write of size 8で

360 217 
0.1 0.0 
0.2 0.0 
0.3 0.0 
0.4 0.0 
0.6 0.0 
0.8 0.0 
1.2 0.0 
2.4 0.0 
4.1 0.0 
8.2 0.0 
12.3 0.0 
16.4 0.0 
24.6 0.0 
32.8 0.0 
49.2 0.0 

Valgrindの。だから私は間違ってメモリを再割り当てしているようだ。しかしもし私がそうであれば、それらを印刷すると値が正常に読み込まれるのはなぜですか？つまり、すべての入力がアレイに正しくロードされます。

私は間違って何をしていますか？ memsetを使用していますか？ freeは正しくありませんか？

Answer 1

メモリを誤って再割り当てしています。 std::malloc、std::callocおよびstd::reallocへのインターフェイスは、割り当てられるサイズに関して多少矛盾し、混乱します。 std::mallocとstd::reallocは単一サイズの引数をとります。これは、オブジェクトのサイズにそのサイズを掛けた値になります。 std::callocは、オブジェクトの数と1つのオブジェクトのサイズの2つの引数をとります。

あなたがこの（再）

auto *tmp = (Tile*)realloc(*array, 2 * sizeof(Tile)); 

を呼び出すには、2枚のタイルのためのスペースを割り当てます。あなたはそれがとても0

Answer 2

void DoubleArray(Tile **array, unsigned *cap) { 
    // 
    auto *tmp = (Tile*)realloc(*array, *cap); 

    if(tmp) { 
    *array = tmp; 
    *cap *= 2; 
    (*cap)--; 
    } else { 
    printf("Error allocating memory.\n"); 
    free(tmp); 
    } 
} 

tmpは（エラーのために）再割り当て、なぜfree()それされていませんか？