diff options
| author | Russ White <russ@riw.us> | 2021-08-03 15:29:09 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-08-03 15:29:09 +0200 |
| commit | cdc549b51acf0452ca34fb50193e6df3442f450a (patch) | |
| tree | 4e3dfa0f5cbe83084bcfb3b8a7c55da290fa6226 /ripd | |
| parent | Merge pull request #9231 from idryzhov/zebra-rmap-set-src (diff) | |
| parent | ripd: fix authentication key length (diff) | |
| download | frr-cdc549b51acf0452ca34fb50193e6df3442f450a.tar.xz frr-cdc549b51acf0452ca34fb50193e6df3442f450a.zip | |
Merge pull request #9191 from idryzhov/rip-auth
ripd: fix authentication key length
Diffstat (limited to 'ripd')
| -rw-r--r-- | ripd/ripd.c | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/ripd/ripd.c b/ripd/ripd.c index ccd4bf1ba..3d1427c3b 100644 --- a/ripd/ripd.c +++ b/ripd/ripd.c @@ -920,9 +920,11 @@ static int rip_auth_md5(struct rip_packet *packet, struct sockaddr_in *from, if (key == NULL || key->string == NULL) return 0; - strlcpy(auth_str, key->string, sizeof(auth_str)); + memcpy(auth_str, key->string, + MIN(sizeof(auth_str), strlen(key->string))); } else if (ri->auth_str) - strlcpy(auth_str, ri->auth_str, sizeof(auth_str)); + memcpy(auth_str, ri->auth_str, + MIN(sizeof(auth_str), strlen(ri->auth_str))); if (auth_str[0] == 0) return 0; @@ -965,9 +967,11 @@ static void rip_auth_prepare_str_send(struct rip_interface *ri, struct key *key, memset(auth_str, 0, len); if (key && key->string) - strlcpy(auth_str, key->string, len); + memcpy(auth_str, key->string, + MIN((size_t)len, strlen(key->string))); else if (ri->auth_str) - strlcpy(auth_str, ri->auth_str, len); + memcpy(auth_str, ri->auth_str, + MIN((size_t)len, strlen(ri->auth_str))); return; } |