Modify

Opened 3 years ago

Last modified 3 years ago

#17934 new defect

dnsmasq init script error

Reported by: Tabis Owned by: developers
Priority: normal Milestone:
Component: packages Version: Trunk
Keywords: Cc:

Description

the init scrpit generates a wrong, and in some cases invalid configfile.

in package/network/services/dnsmasq/files/dnsmasq.init:
append_parm "$cfg" "local" "--server"
should be
append_parm "$cfg" "local" "--local"

also, the list must not only separate parameters by a slash, but also start and end with a slash, as in /para1/para2/para3/

should be simple, but i'm not a coder

Attachments (0)

Change History (1)

comment:1 in reply to: ↑ description Changed 3 years ago by yousong

Replying to Tabis:

the init scrpit generates a wrong, and in some cases invalid configfile.

in package/network/services/dnsmasq/files/dnsmasq.init:
append_parm "$cfg" "local" "--server"
should be
append_parm "$cfg" "local" "--local"

dnsmasq manual says local is a synonym of server for making the config file cleaner.

also, the list must not only separate parameters by a slash, but also start and end with a slash, as in /para1/para2/para3/

yes, but dnsmasq.init does not check this for now. users have the control over validity of the generated dnsmasq conf file.

should be simple, but i'm not a coder

Add Comment

Modify Ticket

Action
as new .
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.