]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
Annotate false positive tainted_argument (CID #1419883) (#4696)
authorJames Jones <jejones3141@gmail.com>
Thu, 25 Aug 2022 20:45:51 +0000 (15:45 -0500)
committerGitHub <noreply@github.com>
Thu, 25 Aug 2022 20:45:51 +0000 (16:45 -0400)
Coverity doesn't see that buffer[0] is checked twice, so
it can never be an invalid subscript for fr_packet_codes[].

src/listen/radius/proto_radius_tcp.c

index 56be90e75f935b3268a5d473278b50087ad89517..002a52600472547c1bb1fce549f11a6123eb7b28 100644 (file)
@@ -192,6 +192,7 @@ static ssize_t mod_read(fr_listen_t *li, UNUSED void **packet_ctx, fr_time_t *re
        /*
         *      Print out what we received.
         */
+       /* coverity[tainted_data] */
        DEBUG2("proto_radius_tcp - Received %s ID %d length %d %s",
               fr_packet_codes[buffer[0]], buffer[1],
               (int) packet_len, thread->name);