diff options
| author | Sree Harsha Totakura <sreeharsha@totakura.in> | 2015-03-26 19:39:16 +0100 |
|---|---|---|
| committer | Sree Harsha Totakura <sreeharsha@totakura.in> | 2015-03-26 19:39:16 +0100 |
| commit | 7730a083494c0d49705f87b32fc85dfa6ed6809a (patch) | |
| tree | de1f65ff2c8e18f7d0acbd5da4576ac1c8e70254 /src/util/amount.c | |
| parent | ffe1ec4cb9c9ad23ecba095acb4e38edb741b402 (diff) | |
util: avoid memcpy over overlapping memory
memcpy used to copy the currency string in TALER_amount_get_zero can copy from
overlapping memory regions. This happens when the diff parameter to
TALER_amount_substract is same as a1 parameter.
Diffstat (limited to 'src/util/amount.c')
| -rw-r--r-- | src/util/amount.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/util/amount.c b/src/util/amount.c index 5e7f69fd9..c13c9101e 100644 --- a/src/util/amount.c +++ b/src/util/amount.c @@ -348,7 +348,7 @@ TALER_amount_subtract (struct TALER_Amount *diff, return GNUNET_SYSERR; } GNUNET_assert (GNUNET_OK == - TALER_amount_get_zero (a1->currency, + TALER_amount_get_zero (n1.currency, diff)); GNUNET_assert (n1.fraction >= n2.fraction); diff->fraction = n1.fraction - n2.fraction; |