Skip to content

Commit 24f7801

Browse files
committed
Pacify Coverity (CID #1619299)
fr_dbuff_in_uint64v doesn't return 0, but Coverity believes it can, which would cause an overflow when populating enc_byte
1 parent eb5a60e commit 24f7801

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/protocols/internal/encode.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ static ssize_t internal_encode(fr_dbuff_t *dbuff,
121121
* integer to the encoding byte.
122122
*/
123123
slen = fr_dbuff_in_uint64v(&work_dbuff, da->attr);
124-
if (slen < 0) return slen;
124+
if (slen <= 0) return slen;
125125
flen = (size_t) slen;
126126
enc_byte |= ((flen - 1) << 5);
127127

0 commit comments

Comments
 (0)