Skip to content

Check user input used as string literals #149

Description

@rra

Reported by iulius on 4 Jul 2015 21:19 UTC
The following programs use format strings that are not string literals at compile time; they come from user input and are not currently sanity-checked.

User input should be checked before being used. We can do something like what the IsValidSubmissionTemplate() function does in lib/getmodaddr.c.

  • batcher: check the values of the arguments given to the -p and -s parameters
  • buffchan: check the value of the argument given to the -d parameter
  • innfeed: check the contents of the timeToStringFormat and deliver_rcpt_to variables (from the log-time-format and deliver-rcpt-to parameters of innfeed.conf)
  • innd: check the contents of the sp->Param variable (from newsfeeds)
  • ninpaths: check the value of the argument given to the -d parameter
  • nnrpd: check the contents of the mta parameter of inn.conf (the check may be useful at other places of the code)

Metadata

Metadata

Assignees

No one assigned

    Labels

    C: generalRelated to several componentsP: lowLow prioritybugSomething isn't working

    Type

    Fields

    No fields configured for Bug.

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions