如何写一个memcpy函数完全MISRA:2012兼容?

时间:2017-10-10 11:38:25

标签: c void-pointers misra qualifiers

我写了这个memcpy函数,但我仍然需要禁用规则11.5和11.8。是否有完整的MISRA:2012兼容解决方案?

#pragma cstat_suppress="MISRAC2012-Rule-21.6" // Uses of stdio.h were found.
#include <stdio.h>

#include <stdint.h>
#include <string.h>
#include <stdlib.h>

extern int main(void);

static int_least8_t _memcpy(void *dst, const void *src, const size_t length)
{
    #pragma cstat_disable="MISRAC2012-Rule-11.5" // A conversion from a pointer to void into a pointer to object was found.
    int_least8_t* destination = (int_least8_t*)dst;
    #pragma cstat_disable="MISRAC2012-Rule-11.8" // A cast that removes a const or volatile qualification was found.
    const int_least8_t* source = (int_least8_t*)src;
    #pragma cstat_restore="MISRAC2012-Rule-11.5","MISRAC2012-Rule-11.8"

    for (size_t i = 0; i < (length / sizeof(int_least8_t)); i++)
    {
        destination[i] = source[i];
    }
    return 0;
}

int main(void)
{
    int32_t src[32];
    int32_t dst[32];

    (void)memset(src, 0xff, sizeof(src));

    (void)_memcpy(dst, src, 128);

    for (size_t i = 0; i < (sizeof(src) / sizeof(src[0])); i++)
    {
        (void)printf("%d <=> %d\n", src[i], dst[i]);
    }

    return 0;
}

我使用IAR作为编译器,使用C-STAT进行静态分析。

1 个答案:

答案 0 :(得分:6)

您无法使用标准格式编写memcpy并且完全符合MISRA标准。你似乎注意到了,MISRA不允许restrict。但也有规则11.5。

规则11.5关于从指针到虚空到指针到类型的强制转换在实践中过于繁琐。这是一个咨询规则,所以我会跳过它。你不需要提出偏差。

关于抛弃限定符的规则11.8然而是合理的(并且是必需的)。在这种情况下,你没有理由这样做。您的代码中存在一个由MISRA阻止的错误。将代码更改为

const int_least8_t* source = (const int_least8_t*) src;

附加说明:

  • 您无需向main()提供前瞻性声明。
  • MISRA-C不允许
  • stdio.h
  • 避免声明以下划线开头的标识符,请参阅C11 7.1.3。
  • 在这里使用int_least8_t没有明显的好处。此外,签名类型存在问题。我会改用uint8_t