summaryrefslogtreecommitdiffstats
path: root/openbsc/src
diff options
context:
space:
mode:
authorHarald Welte <laforge@netfilter.org>2009-08-06 15:44:18 +0200
committerHarald Welte <laforge@netfilter.org>2009-08-06 15:44:18 +0200
commitac606dcf92f2ece21b3e272c34ca5ff8039b2f63 (patch)
tree2875fe401eb3cfa6077943ff399b4be53b3278d2 /openbsc/src
parent4e55d6883b8e0eec853cd5abdf5dbe36142a9d0a (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')
-rw-r--r--openbsc/src/abis_nm.c4
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;
}