clang compiler warnings: Fix -Wbitfield-constant-conversion warning
authorMatthew Jordan <mjordan@digium.com>
Sat, 28 Mar 2015 12:33:21 +0000 (12:33 +0000)
committerMatthew Jordan <mjordan@digium.com>
Sat, 28 Mar 2015 12:33:21 +0000 (12:33 +0000)
In chan_iax2, we attempt to assign a -1 to a bitfield. This gets caught by
clang, as it will truncate the -1 to a 1 implicitly.

Instead, we just assign the value a '1'.

Review: https://reviewboard.asterisk.org/r/4537/

ASTERISK-24917
Reported by: dkdegroot
patches:
  rb4537.patch submitted by dkdegroot (License 6600)
........

Merged revisions 433683 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 433684 from http://svn.asterisk.org/svn/asterisk/branches/13

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

channels/chan_iax2.c

index 4a41bf9..702a763 100644 (file)
@@ -11876,7 +11876,7 @@ immediatedial:
                if (iaxdebug && iaxs[fr->callno]) {
                        ast_debug(1, "Received out of order packet... (type=%u, subclass %d, ts = %u, last = %u)\n", f.frametype, f.subclass.integer, fr->ts, iaxs[fr->callno]->last);
                }
-               fr->outoforder = -1;
+               fr->outoforder = 1;
        }
        fr->cacheable = ((f.frametype == AST_FRAME_VOICE) || (f.frametype == AST_FRAME_VIDEO));
        if (iaxs[fr->callno]) {