diff options
| author | Andrei Emeltchenko <andrei.emeltchenko@intel.com> | 2014-01-29 14:36:00 +0200 |
|---|---|---|
| committer | Luiz Augusto von Dentz <luiz.von.dentz@intel.com> | 2014-01-29 09:05:29 -0800 |
| commit | e315d108283aa60fbd948eff457437f4b8a96252 (patch) | |
| tree | 4715afaeb2b66c7064c616bc91ac483984e5ba1f /android/avctp.c | |
| parent | 3f839fa93506282c96a5e3caae947d2cc0abf597 (diff) | |
| download | bluez-e315d108283aa60fbd948eff457437f4b8a96252.tar.gz | |
android/avctp: Use predefined HEADER_LENGTH instead of sizeof
Make code consistent with using HEADER_LENGTH defined. Remove also type
conversion.
Diffstat (limited to 'android/avctp.c')
| -rw-r--r-- | android/avctp.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/android/avctp.c b/android/avctp.c index 6f9b33bb2..4ebf2ffbe 100644 --- a/android/avctp.c +++ b/android/avctp.c @@ -818,24 +818,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) { |