Opened 13 days ago

#31638 assigned task

config: Always check for valid function arguments at the start of each function

Reported by: teor Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.2.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: network-team-roadmap-august
Cc: nickm, teor, gaba Actual Points:
Parent ID: #29211 Points:
Reviewer: Sponsor:

Description

Some of the new and refactored config functions check if their arguments are NULL (or otherwise valid). Others don't.

We should consistently check for valid arguments at the start of every function. If there are any unusual validity conditions, we should explain them in a function comment.

If we skip validation, we should explain why in a comment, where we would usually do validation.

I'm also open to general exceptions. I'm not sure if we have coding standards here. But it would be nice to catch as many future bugs as we can, as early as we can.

Child Tickets

Change History (0)

Note: See TracTickets for help on using tickets.