clang compiler warnings: Fix format specified in framehook
authorMatthew Jordan <mjordan@digium.com>
Mon, 6 Apr 2015 17:52:31 +0000 (17:52 +0000)
committerMatthew Jordan <mjordan@digium.com>
Mon, 6 Apr 2015 17:52:31 +0000 (17:52 +0000)
This patch fixes an invalid format specifier used in the formatting of an
ERROR message in the framehook code. The format specifier specifies a
type of 'unsigned short', but the argument passed to it is of type 'int'.
The patch changes the format specifier to 'i'.

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

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

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

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

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

main/framehook.c

index 44bb868..e42a232 100644 (file)
@@ -137,7 +137,7 @@ int ast_framehook_attach(struct ast_channel *chan, struct ast_framehook_interfac
        struct ast_framehook_list *fh_list;
        struct ast_frame *frame;
        if (i->version != AST_FRAMEHOOK_INTERFACE_VERSION) {
-               ast_log(LOG_ERROR, "Version '%hu' of framehook interface not what we compiled against (%hu)\n",
+               ast_log(LOG_ERROR, "Version '%hu' of framehook interface not what we compiled against (%i)\n",
                        i->version, AST_FRAMEHOOK_INTERFACE_VERSION);
                return -1;
        }