diff options
author | Harald Welte <laforge@netfilter.org> | 2009-08-06 15:44:18 +0200 |
---|---|---|
committer | Harald Welte <laforge@netfilter.org> | 2009-08-06 15:44:18 +0200 |
commit | ac606dcf92f2ece21b3e272c34ca5ff8039b2f63 (patch) | |
tree | 2875fe401eb3cfa6077943ff399b4be53b3278d2 | |
parent | 4e55d6883b8e0eec853cd5abdf5dbe36142a9d0a (diff) |
use talloc_free() rather than free()
Fix two bugs in OML software download code where we allocate data structures
using talloc, but free() them using the system memory allocator. Spotted by
dexter.
-rw-r--r-- | openbsc/src/abis_nm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/openbsc/src/abis_nm.c b/openbsc/src/abis_nm.c index dec4b2957..8bde6830d 100644 --- a/openbsc/src/abis_nm.c +++ b/openbsc/src/abis_nm.c @@ -2121,7 +2121,7 @@ static int bs11_swload_cbfn(unsigned int hook, unsigned int event, bs11_sw->win_size, bs11_sw->forced, &bs11_swload_cbfn, bs11_sw); - free(fle); + talloc_free(fle); } else { /* activate the SWL */ rc = abis_nm_software_activate(bs11_sw->bts, @@ -2175,7 +2175,7 @@ int abis_nm_bs11_load_swl(struct gsm_bts *bts, const char *fname, /* start download the next file of our file list */ rc = abis_nm_software_load(bts, fle->fname, win_size, forced, bs11_swload_cbfn, bs11_sw); - free(fle); + talloc_free(fle); return rc; } |