Merged revisions 212763 via svnmerge from
authorSean Bright <sean@malleable.com>
Tue, 18 Aug 2009 16:38:36 +0000 (16:38 +0000)
committerSean Bright <sean@malleable.com>
Tue, 18 Aug 2009 16:38:36 +0000 (16:38 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r212763 | seanbright | 2009-08-18 12:36:00 -0400 (Tue, 18 Aug 2009) | 11 lines

  Delay the creation of temporary files until we have a valid manager command to handle.

  Without this patch, asterisk creates a temporary file before determining if the
  specified command is valid.  If invalid, we weren't properly cleaning up the file.

  (closes issue #15730)
  Reported by: zmehmood
  Patches:
        M15730.diff uploaded by junky (license 177)
  Tested by: zmehmood
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@212764 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/manager.c

index f6501c5..e9a9970 100644 (file)
@@ -3126,7 +3126,7 @@ static int action_command(struct mansession *s, const struct message *m)
        const char *id = astman_get_header(m, "ActionID");
        char *buf, *final_buf;
        char template[] = "/tmp/ast-ami-XXXXXX";        /* template for temporary file */
-       int fd = mkstemp(template);
+       int fd;
        off_t l;
 
        if (ast_strlen_zero(cmd)) {
@@ -3139,6 +3139,8 @@ static int action_command(struct mansession *s, const struct message *m)
                return 0;
        }
 
+       fd = mkstemp(template);
+
        astman_append(s, "Response: Follows\r\nPrivilege: Command\r\n");
        if (!ast_strlen_zero(id)) {
                astman_append(s, "ActionID: %s\r\n", id);