]> git.ipfire.org Git - thirdparty/knot-dns.git/commitdiff
libknot: mute some clang-analyzer warnings
authorDaniel Salzman <daniel.salzman@nic.cz>
Tue, 19 Jun 2018 12:47:26 +0000 (14:47 +0200)
committerDaniel Salzman <daniel.salzman@nic.cz>
Tue, 19 Jun 2018 12:47:34 +0000 (14:47 +0200)
src/libknot/rdata.h
src/libknot/wire.h

index 7c0e3facb8eaef3c5acdcba7bd868a608b9d6cff..da191cd8443147105b31ce63689debb62423660c 100644 (file)
@@ -1,4 +1,4 @@
-/*  Copyright (C) 2017 CZ.NIC, z.s.p.o. <knot-dns@labs.nic.cz>
+/*  Copyright (C) 2018 CZ.NIC, z.s.p.o. <knot-dns@labs.nic.cz>
 
     This program is free software: you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -51,6 +51,7 @@ typedef struct {
 inline static void knot_rdata_init(knot_rdata_t *rdata, uint16_t len, const uint8_t *data)
 {
        assert(rdata);
+       assert(data);
        rdata->len = len;
        memcpy(rdata->data, data, len);
 }
index 2bd65702d8c48eebcd8a049c246d97299a73aacc..5f69d9f6e0bd0ec6578981ea372dd939ea3e90f9 100644 (file)
@@ -24,6 +24,7 @@
 
 #pragma once
 
+#include <assert.h>
 #include <stdint.h>
 #include <string.h>
 
@@ -38,6 +39,7 @@
  */
 inline static uint16_t knot_wire_read_u16(const uint8_t *pos)
 {
+       assert(pos);
        uint16_t result;
        memcpy(&result, pos, sizeof(result));
        return be16toh(result);
@@ -52,6 +54,7 @@ inline static uint16_t knot_wire_read_u16(const uint8_t *pos)
  */
 inline static uint32_t knot_wire_read_u32(const uint8_t *pos)
 {
+       assert(pos);
        uint32_t result;
        memcpy(&result, pos, sizeof(result));
        return be32toh(result);
@@ -66,6 +69,7 @@ inline static uint32_t knot_wire_read_u32(const uint8_t *pos)
  */
 inline static uint64_t knot_wire_read_u48(const uint8_t *pos)
 {
+       assert(pos);
        uint64_t input = 0;
        memcpy((uint8_t *)&input + 1, pos, 6);
        return be64toh(input) >> 8;
@@ -80,6 +84,7 @@ inline static uint64_t knot_wire_read_u48(const uint8_t *pos)
  */
 inline static uint64_t knot_wire_read_u64(const uint8_t *pos)
 {
+       assert(pos);
        uint64_t result;
        memcpy(&result, pos, sizeof(result));
        return be64toh(result);
@@ -95,6 +100,7 @@ inline static uint64_t knot_wire_read_u64(const uint8_t *pos)
  */
 inline static void knot_wire_write_u16(uint8_t *pos, uint16_t data)
 {
+       assert(pos);
        uint16_t beval = htobe16(data);
        memcpy(pos, &beval, sizeof(beval));
 }
@@ -109,6 +115,7 @@ inline static void knot_wire_write_u16(uint8_t *pos, uint16_t data)
  */
 inline static void knot_wire_write_u32(uint8_t *pos, uint32_t data)
 {
+       assert(pos);
        uint32_t beval = htobe32(data);
        memcpy(pos, &beval, sizeof(beval));
 }
@@ -123,6 +130,7 @@ inline static void knot_wire_write_u32(uint8_t *pos, uint32_t data)
  */
 inline static void knot_wire_write_u48(uint8_t *pos, uint64_t data)
 {
+       assert(pos);
        uint64_t swapped = htobe64(data << 8);
        memcpy(pos, (uint8_t *)&swapped + 1, 6);
 }
@@ -137,6 +145,7 @@ inline static void knot_wire_write_u48(uint8_t *pos, uint64_t data)
  */
 inline static void knot_wire_write_u64(uint8_t *pos, uint64_t data)
 {
+       assert(pos);
        uint64_t beval = htobe64(data);
        memcpy(pos, &beval, sizeof(beval));
 }