diff options
| author | Eric Laurent <elaurent@google.com> | 2021-01-13 20:58:24 +0100 |
|---|---|---|
| committer | Eric Laurent <elaurent@google.com> | 2021-01-13 21:02:36 +0100 |
| commit | 533365793d0dbbab99e71dc979bd6e0017f2c6db (patch) | |
| tree | 6b7906f540293dbbea8c25435d49da56ce84bdde | |
| parent | 86822e405ffc65736d28cfec1b5934c4a33163ab (diff) | |
| download | tinycompress-533365793d0dbbab99e71dc979bd6e0017f2c6db.tar.gz | |
fix build
Fix build breakage introduced by commit f86d1a2e
with strlcpy argument checks.
Test: make checkbuild
Change-Id: I3084f1e294d15a12923958af9170f00413fae2bb
| -rw-r--r-- | compress_plugin.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/compress_plugin.c b/compress_plugin.c index 24d72ee..7a5538d 100644 --- a/compress_plugin.c +++ b/compress_plugin.c @@ -366,14 +366,15 @@ static int compress_plug_open(unsigned int card, unsigned int device, fprintf(stderr, "%s: invalid library name\n", __func__); goto err_open_fn; } - open_fn = calloc(1, strlen(name) + strlen("_open") + 1); + const size_t open_fn_size = strlen(name) + strlen("_open") + 1; + open_fn = calloc(1, open_fn_size); if (!open_fn) { rc = -ENOMEM; goto err_open_fn; } - strlcpy(open_fn, name, strlen(name) + 1); - strlcat(open_fn, "_open", strlen(name) + strlen("_open") + 1); + strlcpy(open_fn, name, open_fn_size); + strlcat(open_fn, "_open", open_fn_size); plug_data->plugin_open_fn = dlsym(dl_hdl, open_fn); if (!plug_data->plugin_open_fn) { |