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)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

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

    Type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions