diff options
| author | Phil Sutter <phil@nwl.cc> | 2021-03-09 11:26:47 +0100 |
|---|---|---|
| committer | Phil Sutter <phil@nwl.cc> | 2021-03-15 12:23:20 +0100 |
| commit | 960c8e6174374ee113b290c304365bf02ed346d6 (patch) | |
| tree | 3c5b27821e29409822920e6113e1c3364ac5b4e9 /src/expr/redir.c | |
| parent | 4f6e671b462180ba9af12bd30211811b5fca8cb1 (diff) | |
Drop pointless local variable in snprintf callbacks
A common idiom among snprintf callbacks was to copy the unsigned
parameter 'size' (or 'len') into a signed variable for further use.
Though since snprintf() itself casts it to unsigned and
SNPRINTF_BUFFER_SIZE() does not allow it to become negative, this is not
needed. Drop the local variable and rename the parameter accordingly.
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'src/expr/redir.c')
| -rw-r--r-- | src/expr/redir.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/expr/redir.c b/src/expr/redir.c index 8de4c60..4f56cb4 100644 --- a/src/expr/redir.c +++ b/src/expr/redir.c @@ -132,10 +132,10 @@ nftnl_expr_redir_parse(struct nftnl_expr *e, struct nlattr *attr) } static int -nftnl_expr_redir_snprintf(char *buf, size_t len, +nftnl_expr_redir_snprintf(char *buf, size_t remain, uint32_t flags, const struct nftnl_expr *e) { - int ret, remain = len, offset = 0; + int ret, offset = 0; struct nftnl_expr_redir *redir = nftnl_expr_data(e); if (nftnl_expr_is_set(e, NFTNL_EXPR_REDIR_REG_PROTO_MIN)) { |