Opened 3 months ago

Last modified 3 months 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.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: network-team-roadmap-august, 042-deferred-20190918
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 (1)

comment:1 Changed 3 months ago by nickm

Keywords: 042-deferred-20190918 added
Milestone: Tor: 0.4.2.x-finalTor: 0.4.3.x-final

Defer numerous 0.4.2 tickets to 0.4.3.

Note: See TracTickets for help on using tickets.