From ec1d828ebdbb51c846a802db0507861584528dc7 Mon Sep 17 00:00:00 2001 From: Andrei Emeltchenko Date: Wed, 29 Jan 2014 14:36:04 +0200 Subject: [PATCH] avctp: Use predefined HEADER_LENGTH instead of sizeof Make code consistent with using HEADER_LENGTH already defined and used instead of sizeof(). Remove also ugly type conversion. --- profiles/audio/avctp.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/profiles/audio/avctp.c b/profiles/audio/avctp.c index 570d60994..0a1a4572a 100644 --- a/profiles/audio/avctp.c +++ b/profiles/audio/avctp.c @@ -924,24 +924,24 @@ static gboolean session_cb(GIOChannel *chan, GIOCondition cond, gpointer data) if (ret <= 0) goto failed; - if ((unsigned int) ret < sizeof(struct avctp_header)) { + if (ret < AVCTP_HEADER_LENGTH) { error("Too small AVCTP packet"); goto failed; } avctp = (struct avctp_header *) buf; - ret -= sizeof(struct avctp_header); - if ((unsigned int) ret < sizeof(struct avc_header)) { + ret -= AVCTP_HEADER_LENGTH; + if (ret < AVC_HEADER_LENGTH) { error("Too small AVCTP packet"); goto failed; } - avc = (struct avc_header *) (buf + sizeof(struct avctp_header)); + avc = (struct avc_header *) (buf + AVCTP_HEADER_LENGTH); - ret -= sizeof(struct avc_header); + ret -= AVC_HEADER_LENGTH; - operands = (uint8_t *) avc + sizeof(struct avc_header); + operands = (uint8_t *) avc + AVC_HEADER_LENGTH; operand_count = ret; if (avctp->cr == AVCTP_RESPONSE) { -- 2.47.3