[coreboot-gerrit] Change in ...coreboot[master]: lib.h: Add convenient memcpy() variants

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32740 )

Change subject: lib.h: Add convenient memcpy() variants
......................................................................

Patch Set 1:

(1 comment)
> Patch Set 1:
>
> Wondering if this is something that would get general
> approval and adoption?
>
> I tried to find a way to check if somebody passes
> a pointer by accident, but that doesn't seem easy
> to do.

https://review.coreboot.org/#/c/32740/1/src/include/lib.h 
File src/include/lib.h:

https://review.coreboot.org/#/c/32740/1/src/include/lib.h@74 
PS1, Line 74: #define memcpy_dlen(dst, src) memcpy(dst, src, sizeof(dst))
Isn't this expecting pointers for src and dst? If not, shouldn't these be &dst and &src?