Broken check for empty "bridge-ips" line in validate_bridge_stats()
geoip_format_bridge_stats() generates the bridge-stats file contents, and if country_data is NULL then it writes "bridge-ips \n" to the file.
Then when validate_bridge_stats() is called, if it can't find a populated bridge-ips line it tries to match "bridge-ips\n", which should always fail because of the extra whitespace that was added in geoip_format_bridge_stats(). Isn't that right?
The only thing that confuses me is why this hasn't triggered so far. Maybe something in my analysis is incorrect, or the bridge-stats file is always generated after we accept a connection?
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information