]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: server: set rid default value in new_server()
authorAurelien DARRAGON <adarragon@haproxy.com>
Fri, 7 Jul 2023 13:19:36 +0000 (15:19 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 10 Jul 2023 16:28:08 +0000 (18:28 +0200)
commitb2f70694797c6a3986f57584d6230d668e961651
tree742a85887d964cf6cfbb5f1ccaa1b31301e07b80
parent9859e00981fa29d6ab8dfa5bfc479aefb0a1bcf5
BUG/MINOR: server: set rid default value in new_server()

srv->rid default value is set in _srv_parse_init() after the server is
succesfully allocated using new_server().

This is wrong because new_server() can be used independently so rid value
assignment would be skipped in this case.

Hopefully new_server() allocates server data using calloc() so srv->rid
is already set to 0 in practise. But if calloc() is replaced by malloc()
or other memory allocating function that doesn't zero-initialize srv
members, this could lead to rid being uninitialized in some cases.

This should be backported in 2.8 with 61e3894dfe ("MINOR: server: add
srv->rid (revision id) value")
src/server.c