diff options
| author |   krw <krw@openbsd.org> | 2009-08-25 10:42:27 +0000 |
|---|---|---|
| committer | krw <krw@openbsd.org> | 2009-08-25 10:42:27 +0000 |
| commit | 5fd13239c8237c619960a3500c89ffb22bc4acb3 (patch) | |
| tree | 8947ccf4a4a7ca3f360967ab4d19a9ac7ce52555 | |
| parent | djm accidentally changed the type of dirfilt in the prototypes (diff) | |
| download | wireguard-openbsd-5fd13239c8237c619960a3500c89ffb22bc4acb3.tar.xz wireguard-openbsd-5fd13239c8237c619960a3500c89ffb22bc4acb3.zip | |
Fix order of memcpy() arguments so data is copied from src to dest,
not the other way. At least partial fix for interfaces with >64
aliases. From Max Laier via Marc Balmer.
'seems right' sthen@ ok markus@
| -rw-r--r-- | sys/net/pf_if.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net/pf_if.c b/sys/net/pf_if.c index 6a29e361025..d8c99e44c0a 100644 --- a/sys/net/pf_if.c +++ b/sys/net/pf_if.c @@ -1,4 +1,4 @@ -/* $OpenBSD: pf_if.c,v 1.56 2009/03/09 13:53:10 mcbride Exp $ */ +/* $OpenBSD: pf_if.c,v 1.57 2009/08/25 10:42:27 krw Exp $ */ /* * Copyright 2005 Henning Brauer <henning@openbsd.org> @@ -529,7 +529,7 @@ pfi_address_add(struct sockaddr *sa, int af, int net) "(%d/%d)\n", pfi_buffer_cnt, PFI_BUFFER_MAX); return; } - memcpy(pfi_buffer, p, pfi_buffer_cnt * sizeof(*pfi_buffer)); + memcpy(p, pfi_buffer, pfi_buffer_cnt * sizeof(*pfi_buffer)); /* no need to zero buffer */ free(pfi_buffer, PFI_MTYPE); pfi_buffer = p; |