[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-bugs] #8051 [Tor]: Broken check for empty "bridge-ips" line in validate_bridge_stats()
#8051: Broken check for empty "bridge-ips" line in validate_bridge_stats()
------------------------+---------------------------------------------------
Reporter: asn | Owner:
Type: defect | Status: new
Priority: normal | Milestone: Tor: 0.2.4.x-final
Component: Tor | Version:
Keywords: tor-bridge | Parent:
Points: | Actualpoints:
------------------------+---------------------------------------------------
`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?
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/8051>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs