diff options
author | cbdev <cb@cbcdn.com> | 2021-06-23 22:27:15 +0200 |
---|---|---|
committer | cbdev <cb@cbcdn.com> | 2021-06-23 22:27:15 +0200 |
commit | a89dcd7942958e0ebe5881dde4820ceb77d29e9b (patch) | |
tree | a10e8de8265b61255a41261f0bfb6bceb40938f5 /backends/rtpmidi.c | |
parent | 6cfb87b2e1ce8e77563fdc9ac9b234808ad5acd0 (diff) | |
download | midimonster-a89dcd7942958e0ebe5881dde4820ceb77d29e9b.tar.gz midimonster-a89dcd7942958e0ebe5881dde4820ceb77d29e9b.tar.bz2 midimonster-a89dcd7942958e0ebe5881dde4820ceb77d29e9b.zip |
Fix build with recent environments, fix Coverity CID 371602 and 355842
Diffstat (limited to 'backends/rtpmidi.c')
-rw-r--r-- | backends/rtpmidi.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/backends/rtpmidi.c b/backends/rtpmidi.c index f0987f2..922075e 100644 --- a/backends/rtpmidi.c +++ b/backends/rtpmidi.c @@ -1329,13 +1329,14 @@ static int rtpmidi_mdns_detach(instance* inst){ } offset += bytes; - //TODO length-checks here - frame[offset++] = strlen(inst->name); - memcpy(frame + offset, inst->name, strlen(inst->name)); - offset += strlen(inst->name); + //calculate maximum permitted instance name length + bytes = min(min(strlen(inst->name), sizeof(frame) - offset - 3), 255); + frame[offset++] = bytes; + memcpy(frame + offset, inst->name, bytes); + offset += bytes; frame[offset++] = 0xC0; frame[offset++] = sizeof(dns_header); - rr->data = htobe16(1 + strlen(inst->name) + 2); + rr->data = htobe16(1 + bytes + 2); free(name.name); return rtpmidi_mdns_broadcast(frame, offset); |