Uploaded image for project: 'FreeSWITCH'
  1. FreeSWITCH
  2. FS-8328

Consider inspecting the application's logic. It's possible that 'else' keyword is missing. conference_api.c 1532

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: mod_conference
    • Labels:
      None
    • CPU Architecture:
      x86-64
    • Kernel:
      Linux
    • Userland:
      GNU/Linux
    • Distribution:
      Debian
    • Distribution Version:
      Debian 8 jessie
    • Compiler:
      gcc
    • FreeSWITCH GIT Revision:
      a5a3561b2b730bca6e46ecf865b26d561ffca704
    • GIT Master Revision hash::
      a5a3561b2b730bca6e46ecf865b26d561ffca704

      Description

      switch_status_t conference_api_sub_vid_logo_img(....)
      {
        ....
        if (!strcasecmp(text, "allclear")) {
          switch_channel_set_variable(member->channel, "....", NULL);
          member->video_logo = NULL;
        } if (!strcasecmp(text, "clear")) { //<==
          member->video_logo = NULL;
        } else {
          member->video_logo = switch_core_strdup(member->pool, text);
        }
        ....
      }

        Attachments

          Activity

            People

            • Assignee:
              brian@freeswitch.org Brian West
              Reporter:
              brian@freeswitch.org Brian West
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: