[Home]

Summary:ASTERISK-16285: [patch] Wrong check for privilege in Originate action
Reporter:Richard Kenner (kenner)Labels:
Date Opened:2010-06-23 21:36:06Date Closed:2010-06-29 20:07:57
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/ManagerInterface
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) manager.c.diff
Description:The test for the 'System' application in action_originate is backwards.
Comments:By: Tzafrir Cohen (tzafrir) 2010-06-29 13:45:07

Why not apply the same fix for the nearby test for Exec?
By: Richard Kenner (kenner) 2010-06-29 19:59:17

I don't understand the question.  The test for Exec looks right to me.
By: Digium Subversion (svnbot) 2010-06-29 20:07:01

Repository: asterisk
Revision: 273144

U   trunk/main/manager.c

------------------------------------------------------------------------
r273144 | tilghman | 2010-06-29 20:07:01 -0500 (Tue, 29 Jun 2010) | 8 lines

Permission checking for the system application is backwards.

(closes issue ASTERISK-16285)
Reported by: kenner
Patches:
      manager.c.diff uploaded by kenner (license 1040)
Tested by: kenner

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=273144
By: Digium Subversion (svnbot) 2010-06-29 20:07:56

Repository: asterisk
Revision: 273145

_U  branches/1.6.2/
U   branches/1.6.2/main/manager.c

------------------------------------------------------------------------
r273145 | tilghman | 2010-06-29 20:07:56 -0500 (Tue, 29 Jun 2010) | 15 lines

Merged revisions 273144 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r273144 | tilghman | 2010-06-29 20:07:02 -0500 (Tue, 29 Jun 2010) | 8 lines
 
 Permission checking for the system application is backwards.
 
 (closes issue ASTERISK-16285)
  Reported by: kenner
  Patches:
        manager.c.diff uploaded by kenner (license 1040)
  Tested by: kenner
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=273145