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 /openbsc/src/abis_nm.c | |
| 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.
Diffstat (limited to 'openbsc/src/abis_nm.c')
| -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; } |