Memcpy() in secure programming?

前端 未结 10 2073
天命终不由人
天命终不由人 2020-12-08 16:34

I recently stumbled across an article that claims Microsoft is banning the memcpy() function in its secure programming shops. I understand the vulnerabilities i

相关标签:
10条回答
  • 2020-12-08 16:53

    The article itself describes a safer alternative: memcpy_s, which requires you to specify the maximum length of the target. When that number is provided independent of the amount of bytes to copy, it acts as a barrier to prevent buffer overflow. Of course, you can abuse that by giving the same number to both.

    0 讨论(0)
  • 2020-12-08 16:54

    I think that C should leave an option to the programmer to shoot his own foot. manual memory management is safe if done properly.

    0 讨论(0)
  • 2020-12-08 16:56

    The alternative is to call memcpy_s

    0 讨论(0)
  • 2020-12-08 16:58

    Microsoft provides alternatives to memcpy and wmemcpy that validate their parameters.

    memcpy_s says, "Hmm, before I read from this address, let me verify for myself that it is not a null pointer; and before I write to this address, I shall perform that test again. I shall also compare the number of bytes I have been requested to copy to the claimed size of the destination; if and only if the call passes all these tests shall I perform the copy."

    memcpy says "Stuff the destination into a register, stuff the source into a register, stuff the count into a register, perform MOVSB or MOVSW." (Example on geocities, not long for this world: http://www.geocities.com/siliconvalley/park/3230/x86asm/asml1013.html)

    Edit: For an example in the wild of the Your Wish Is My Command approach to memcpy, consider OpenSolaris, where memcpy is (for some configurations) defined in terms of bcopy, and bcopy (for some configurations) is ...

    void
         33 bcopy(from, to, count)
         34 #ifdef vax
         35     unsigned char *from, *to;
         36     int count;
         37 {
         38 
         39     asm("   movc3   12(ap),*4(ap),*8(ap)");
         40 }
         41 #else
         42 #ifdef u3b      /* movblkb only works with register args */
         43     unsigned char *from, *to;
         44     int count;
         45 {
         46     asm("   movblkb %r6, %r8, %r7");
         47 }
         48 #else
         49     unsigned char *from, *to;
         50     int count;
         51 {
         52     while ((count--) > 0)
         53         *to++ = *from++;
         54 }
         55 #endif
    

    Edit: Thanks, Millie Smith! Here is what was on the geocities page I linked above:

    MOVS

    The instruction movs is used to copy source string into the destination (yes, copy, not move). This instruction has two variants: movsb and movsw. The movsb ("move string byte") moves one byte at a time, whereas movsw moves two bytes at a time.

    Since we'd like to move several bytes at a time, these movs instructions are done in batches using rep prefix. The number of movements is specified by CX register. See the example below:

    :
    lds   si, [src]
    les   di, [dest]
    cld
    mov   cx, 100
    rep   movsb
    :
    

    This example will copy 100 bytes from src to dest. If you replace movsb with movsw, you copy 200 bytes instead. If you remove the rep prefix, the CX register will have no effect. You will move one byte (if it is movsb, or 2 bytes if it is movsw).

    0 讨论(0)
提交回复
热议问题