summaryrefslogtreecommitdiffstats
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
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.
-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;
}