当动态分配数组的函数被多次调用时,malloc错误?

时间:2012-05-08 18:37:13

标签: c++ arrays heap

我有一个生成价格晶格的函数和一个使用该晶格为相关选项定价的函数。 generate_lattice函数为数组动态分配空间,并返回指向该数组的指针。该指针是传递给price_from_tree的指针。这是代码:

#include <iostream>
#include <math.h>

using std::cout;
using std::endl;
using std::max;
using std::nothrow;

double* generate_lattice(double asset_price, double u, double d, int periods)
{
    int nodes = (periods * (periods + 1)) / 2; //Number of nodes = sum(1, # of periods)
    double * lattice; 

    lattice = new (nothrow) double[nodes]; 
    double up, down;
    int price_index = 0;
    for (int period = 0; period < periods + 1; period++) {
        for (int exp = 0; exp < period + 1; exp++) {
            up = pow(1 + u, exp);
            down = pow(1 - d, period - exp);
            lattice[price_index] = asset_price * up * down;
            price_index++;
        }
    }
    return lattice;
}

double price_from_tree(double* lattice, int periods, double strike, double risk_free, double period_len_years, bool euro = true, bool call = true)
{
    int num_prices = ((periods + 1) * (periods + 2)) / 2;
    double asset_max = lattice[num_prices - 1];
    double asset_min = lattice[num_prices - (periods + 1)];
    double asset_t0 = lattice[0];
    double u = pow(asset_max / asset_t0, pow(periods, -1));
    double d = pow(asset_min / asset_t0, pow(periods, -1));
    double p = (exp(risk_free * period_len_years) - d) / (u - d);
    double p_minus1 = 1 - p;
    int start_node;
    if (euro == true) { start_node = num_prices - periods - 1; }
    else { start_node = 0; }
    int sign;
    if (call == true) { sign = 1; }
    else { sign = -1; }
    for (int node = start_node; node < num_prices; node++) {
        lattice[node] = max(sign * (lattice[node] - strike), 0.0);
    }
    int price_index = num_prices - 1;
    double pv_mult = exp(-risk_free * period_len_years);
    double down_payoff, up_payoff, prev_payoff;
    for (int period = periods + 1; period > 0; period--) {
        for (int node = 0; node < period - 1; node++) {
            down_payoff = lattice[price_index - (node + 1)];
            up_payoff = lattice[price_index - node];
            prev_payoff = pv_mult * (p * up_payoff + p_minus1 * down_payoff);
            if (euro == false) {
                prev_payoff = max(lattice[price_index - (node + 1) - (period - 1)], prev_payoff);
            }
            lattice[price_index - (node + 1) - (period - 1)] = prev_payoff;
        }
        price_index -= period;
    }
    return lattice[0];
}

int main(int argc, char** argv)
{
    double* lattice = generate_lattice(100, 0.10, 0.10, 2);
    double option1 = price_from_tree(lattice, 2, 105, 0.05, 1, true, true);
    cout<<"option 1: "<<option1<<endl;
    double* lattice2 = generate_lattice(100, 0.10, 0.10, 2);
    return 0;
}

当我运行代码时,我得到了这个输出:

  

选项1:8.28214       test:malloc.c:3096:sYSMALLOc:断言`(old_top ==(((mbinptr)(((char *)&amp;((av) - &gt; bins [((1) - 1)* 2])) - __builtin_offsetof(struct   malloc_chunk,fd))))&amp;&amp; old_size == 0)|| ((unsigned long)(old_size)

     
    

=(unsigned long)(((__ builtin_offsetof(struct malloc_chunk,fd_nextsize))+((2 *(sizeof(size_t))) - 1))&amp;〜((2 *(sizeof(size_t)))      - 1)))&amp;&amp; ((old_top) - &gt; size&amp; 0x1)&amp;&amp; ((unsigned long)old_end&amp; pagemask)== 0)&#39;失败。         中止

  
------------------
(program exited with code: 134)

我在错误134上找到的所有内容都是The job is killed with an abort signal.的描述。在我尝试的每种情况下,代码都会为price_from_tree返回正确的值,但如果我包含多个调用generate_lattice它因所述错误而失败。我的price_from_tree函数是否存在导致内存混乱的问题?在这种情况下,我最好使用向量吗?

5 个答案:

答案 0 :(得分:3)

你正在腐败你的堆非常糟糕。如果你使用valgrind运行(假设你在Linux机器上),那就有很多错误。

==23357== Invalid write of size 8
==23357==    at 0x400B9E: generate_lattice(double, double, double, int) (so.cpp:21)
==23357==    by 0x400FC4: main (so.cpp:68)
==23357==  Address 0x595b058 is 0 bytes after a block of size 24 alloc'd
==23357==    at 0x4C27FFB: operator new[](unsigned long, std::nothrow_t const&) (vg_replace_malloc.c:325)
==23357==    by 0x400B1A: generate_lattice(double, double, double, int) (so.cpp:14)
==23357==    by 0x400FC4: main (so.cpp:68)

答案 1 :(得分:1)

你的数学已关闭。在generate_latticeprice_index增长超过(periods * (periods + 1)) / 2。尝试:

int nodes = ((periods+1) * (periods + 2)) / 2; //Number of nodes = sum(1, # of periods + 1)

答案 2 :(得分:1)

共享1维数组lattice的索引计算在两个函数中不一致:

(periods * (periods + 1)) / 2;       // generate_lattice
((periods + 1) * (periods + 2)) / 2; // price_from_tree 

这可能是相关malloc错误的来源。

答案 3 :(得分:1)

generate_lattice创建一个大小为n *(n + 1)/ 2的数组,但是然后初始化该数组的(n + 1)*(n + 1)个元素,对吗?这可能会破坏堆或类似的东西。

我没有看price_from_tree,但如果您只是使用lattice作为数组,我怀疑它会导致问题(除非你走到最后)。

答案 4 :(得分:0)

您正在错误地计算节点数,因此您正在运行数组的末尾。尝试使用句点= 2.因为你正在做&lt;句点+ 1,您必须将节点计算为(句点+ 1)*(句点+ 2)/ 2