fix use of arg_cnt vs arg_len

diff --git a/src/protocols/tacacs/encode.c b/src/protocols/tacacs/encode.c
index 24737e38d8d38698fe265838d1aa46f78345ab53..15f610f92a7fd071918ead16e10681d3c689cc58 100644 (file)
--- a/src/protocols/tacacs/encode.c
+++ b/src/protocols/tacacs/encode.c
@@ -70,10 +70,10 @@ static uint8_t tacacs_encode_body_arg_cnt(fr_pair_list_t *vps, fr_dict_attr_t co
        return arg_cnt;
 }
 
-static ssize_t tacacs_encode_body_arg_n(fr_dbuff_t *dbuff, uint8_t *arg_cnt_p, fr_pair_list_t *vps, fr_dict_attr_t const *da)
+static ssize_t tacacs_encode_body_arg_n(fr_dbuff_t *dbuff, uint8_t arg_cnt, uint8_t *arg_len, fr_pair_list_t *vps, fr_dict_attr_t const *da)
 {
        fr_pair_t   *vp;
-       uint8_t     arg_cnt = 0;
+       uint8_t     i = 0;
        fr_dbuff_t  work_dbuff = FR_DBUFF(dbuff);
 
        for (vp = fr_pair_list_head(vps);
@@ -81,8 +81,8 @@ static ssize_t tacacs_encode_body_arg_n(fr_dbuff_t *dbuff, uint8_t *arg_cnt_p, f
             vp = fr_pair_list_next(vps,vp)) {
                int len;
 
-               if (arg_cnt == 255) break;
-               if (arg_cnt > *arg_cnt_p) break;
+               if (i == 255) break;
+               if (i > arg_cnt) break;
 
                if (vp->da->flags.internal) continue;
 
@@ -95,7 +95,7 @@ static ssize_t tacacs_encode_body_arg_n(fr_dbuff_t *dbuff, uint8_t *arg_cnt_p, f
                        /* Append the <arg_N> field */
                        FR_DBUFF_IN_MEMCPY_RETURN(&work_dbuff, vp->vp_strvalue, vp->vp_length);
 
-                       FR_PROTO_TRACE("arg[%d] --> %s", arg_cnt, vp->vp_strvalue);
+                       FR_PROTO_TRACE("arg[%d] --> %s", i, vp->vp_strvalue);
                        len = vp->vp_length;
 
                } else if (fr_dict_by_da(vp->da) != dict_tacacs) {
@@ -124,14 +124,13 @@ static ssize_t tacacs_encode_body_arg_n(fr_dbuff_t *dbuff, uint8_t *arg_cnt_p, f
 
                        len = fr_dbuff_used(&arg_dbuff);
 
-                       FR_PROTO_TRACE("arg[%d] --> %.*s", arg_cnt, len, fr_dbuff_start(&arg_dbuff));
+                       FR_PROTO_TRACE("arg[%d] --> %.*s", i, len, fr_dbuff_start(&arg_dbuff));
 
                        fr_dbuff_set(&work_dbuff, &arg_dbuff);
                }
 
-               FR_PROTO_TRACE("len(arg[%d]) = %d", arg_cnt, len);
-               arg_cnt_p[1 + arg_cnt] = len;
-               arg_cnt++;
+               FR_PROTO_TRACE("len(arg[%d]) = %d", i, len);
+               arg_len[i++] = len;
        }
 
        return fr_dbuff_set(dbuff, &work_dbuff);
@@ -471,7 +470,7 @@ ssize_t fr_tacacs_encode(fr_dbuff_t *dbuff, uint8_t const *original_packet, char
                         *      Append 'args_body' to the end of buffer
                         */
                        if (packet->author.req.arg_cnt > 0) {
-                               if (tacacs_encode_body_arg_n(&work_dbuff, &packet->author.req.arg_cnt, vps, attr_tacacs_argument_list) < 0) goto error;
+                               if (tacacs_encode_body_arg_n(&work_dbuff, packet->author.req.arg_cnt, &packet->author.req.arg_len[0], vps, attr_tacacs_argument_list) < 0) goto error;
                        }
 
                        goto check_request;
@@ -542,7 +541,7 @@ ssize_t fr_tacacs_encode(fr_dbuff_t *dbuff, uint8_t const *original_packet, char
                         *      Append 'args_body' to the end of buffer
                         */
                        if (packet->author.res.arg_cnt > 0) {
-                               if (tacacs_encode_body_arg_n(&work_dbuff, &packet->author.res.arg_cnt, vps, attr_tacacs_argument_list) < 0) goto error;
+                               if (tacacs_encode_body_arg_n(&work_dbuff, packet->author.res.arg_cnt, &packet->author.res.arg_len[0], vps, attr_tacacs_argument_list) < 0) goto error;
                        }
 
                        goto check_reply;
@@ -612,7 +611,7 @@ ssize_t fr_tacacs_encode(fr_dbuff_t *dbuff, uint8_t const *original_packet, char
                         *      Append 'args_body' to the end of buffer
                         */
                        if (packet->acct.req.arg_cnt > 0) {
-                               if (tacacs_encode_body_arg_n(&work_dbuff, &packet->acct.req.arg_cnt, vps, attr_tacacs_argument_list) < 0) goto error;
+                               if (tacacs_encode_body_arg_n(&work_dbuff, packet->acct.req.arg_cnt, &packet->acct.req.arg_len[0], vps, attr_tacacs_argument_list) < 0) goto error;
                        }
 
                check_request:

diff --git a/src/protocols/tacacs/tacacs.h b/src/protocols/tacacs/tacacs.h
index dbb6caa0641331c7a91dcb50403882561ab53625..e7740c2f539fdb7b0134299d5ac8cd0e0b2262ab 100644 (file)
--- a/src/protocols/tacacs/tacacs.h
+++ b/src/protocols/tacacs/tacacs.h
@@ -195,6 +195,7 @@ typedef struct CC_HINT(__packed__) {
        uint8_t                         port_len;
        uint8_t                         rem_addr_len;
        uint8_t                         arg_cnt;
+       uint8_t                         arg_len[];
 } fr_tacacs_packet_author_req_hdr_t;
 
 typedef enum {
@@ -210,6 +211,7 @@ typedef struct CC_HINT(__packed__) {
        uint8_t                         arg_cnt;
        uint16_t                        server_msg_len;
        uint16_t                        data_len;
+       uint8_t                         arg_len[];
 } fr_tacacs_packet_author_res_hdr_t;
 
 typedef enum {
@@ -228,6 +230,7 @@ typedef struct CC_HINT(__packed__) {
        uint8_t                         port_len;
        uint8_t                         rem_addr_len;
        uint8_t                         arg_cnt;
+       uint8_t                         arg_len[];
 } fr_tacacs_packet_acct_req_hdr_t;
 
 typedef enum {