[mythtv-commits] Ticket #10677: Safely escape shell arguments

MythTV noreply at mythtv.org
Tue May 1 05:12:30 UTC 2012


#10677: Safely escape shell arguments
-------------------------+----------------------------------
 Reporter:  github@…     |           Type:  Patch - Bug Fix
   Status:  new          |       Priority:  minor
Milestone:               |      Component:  MythTV - General
  Version:  Master Head  |       Severity:  medium
 Keywords:               |  Ticket locked:  0
-------------------------+----------------------------------
 The existing ShellEscape method(s) are not consistent and not thorough.
 This implementation will handle spaces, single quotes, double quotes,
 newlines, and all other shell metacharacters.

 An attempt was made to fix this in changeset
 f4dbb71f4e974bc388d4c6f110e6936de9cf490b, (issue #9913), but it didn't
 escape most characters. For example, it didn't address newlines or pipes.
 It also didn't work with arguments that contained both a space and a
 quote.

 The patch is in GitHub: https://github.com/MythTV/mythtv/pull/18

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10677>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center


More information about the mythtv-commits mailing list