[mythtv-commits] Ticket #7817: Add quoting to the location passed to the weather parsers

[MythTV]

#7817: Add quoting to the location passed to the weather parsers
---------------------------------+------------------------------------------
 Reporter:  knight@…             |        Owner:  kormoc 
     Type:  patch                |       Status:  new    
 Priority:  trivial              |    Milestone:  unknown
Component:  Plugin - MythWeb     |      Version:  head   
 Severity:  low                  |   Resolution:         
  Mlocked:  0                    |  
---------------------------------+------------------------------------------
Changes (by Miguel Yarza <miguel.yarza@…>):

  * status:  closed => new
  * resolution:  fixed =>


Comment:

 There is an error in the !GetData function runScript argument escaping,
 last two arguments are being escaped together.

 I got it working escaping arguments separately:

 {{{
 $output_array = $this->runScript($script, '-u '. escapeshellarg($units)
 .' -d '. escapeshellarg(getcwd() .'/'. data) .' '.
 escapeshellarg($location));
 }}}

 I attach a diff against [24932] with the fix.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/7817#comment:4>
MythTV <http://www.mythtv.org/>
MythTV