git: b50a4dce1854 - main - libalias: Avoid uninitialized expiration

Lutz Donnerhacke donner at FreeBSD.org
Fri Jul 2 23:09:28 UTC 2021


The branch main has been updated by donner:

URL: https://cgit.FreeBSD.org/src/commit/?id=b50a4dce185481bebf8096c27588ae04a4a2fd7c

commit b50a4dce185481bebf8096c27588ae04a4a2fd7c
Author:     Lutz Donnerhacke <donner at FreeBSD.org>
AuthorDate: 2021-07-02 23:02:53 +0000
Commit:     Lutz Donnerhacke <donner at FreeBSD.org>
CommitDate: 2021-07-02 23:09:18 +0000

    libalias: Avoid uninitialized expiration
    
    The expiration time of direct address mappings is explicitly
    uninitialized.  Expire times are always compared during housekeeping.
    Despite the uninitialized value does not harm, it's simpler to just
    set it to a reasonable default.  This was detected during valgrinding
    the test suite.
    
    MFC after:      3 days
---
 sys/netinet/libalias/alias_db.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/sys/netinet/libalias/alias_db.c b/sys/netinet/libalias/alias_db.c
index 19529271f49f..9f8c6064d2a7 100644
--- a/sys/netinet/libalias/alias_db.c
+++ b/sys/netinet/libalias/alias_db.c
@@ -613,8 +613,6 @@ AddLink(struct libalias *la, struct in_addr src_addr, struct in_addr dst_addr,
 	case LINK_FRAGMENT_PTR:
 		lnk->expire.time = FRAGMENT_PTR_EXPIRE_TIME;
 		break;
-	case LINK_ADDR:
-		break;
 	default:
 		lnk->expire.time = PROTO_EXPIRE_TIME;
 		break;


More information about the dev-commits-src-all mailing list