Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

The example below, is inspired by Dowd, assuming assumes that the size of struct buffer is equal to the sizeof(size_t) + (sizeof(char) * 50), which (assuming sizeof(size_t) is 4 bytes) is 54 may be incorrect. The size of struct buffer may actually be a larger due to structure padding.

Code Block
struct buffer {
    size_t size;
    char buffer[50];
};

...

void func(struct buffer *buf) {

  struct buffer *buf_cpy = malloc((sizeof(size_t)+(sizeof(char)*50))); /* assuming sizeof(size_t) is 4, this equals 54 */
  if (buf_cpy == NULL) {
    /* Handle malloc() error */
  }

  ...

  memcpy(buf_cpy, buf, sizeof(struct buffer)); /* May result with padding, sizeof(struct buffer) may equal 56, causing in a small buffer overflow */
}

...