diff options
| author | Daniel P. Berrange <berrange@redhat.com> | 2012-03-13 16:39:45 +0000 |
|---|---|---|
| committer | Daniel P. Berrange <berrange@redhat.com> | 2012-04-25 09:42:15 +0100 |
| commit | 7af613362a2e320574dc688fad98294e90f029cc (patch) | |
| tree | 7a147b43bfd14d435ca1f935a7fd07bff807668f /server/smartcard.c | |
| parent | 84c7a95c22ba3a0fc5623c7cd66c9793e02ebcf8 (diff) | |
| download | spice-7af613362a2e320574dc688fad98294e90f029cc.tar.gz spice-7af613362a2e320574dc688fad98294e90f029cc.tar.xz spice-7af613362a2e320574dc688fad98294e90f029cc.zip | |
VSCMsgHeader.reader_id is unsigned, so don't check for >= 0
* server/smartcard.c: Fix assertion to not check for >= 0 on
unsigned variable
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Diffstat (limited to 'server/smartcard.c')
| -rw-r--r-- | server/smartcard.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/server/smartcard.c b/server/smartcard.c index 805cb6e1..eb2823aa 100644 --- a/server/smartcard.c +++ b/server/smartcard.c @@ -428,8 +428,7 @@ static void smartcard_channel_write_to_reader(VSCMsgHeader *vheader) uint32_t n; uint32_t actual_length = vheader->length; - spice_assert(vheader->reader_id >= 0 && - vheader->reader_id <= g_smartcard_readers.num); + spice_assert(vheader->reader_id <= g_smartcard_readers.num); sin = g_smartcard_readers.sin[vheader->reader_id]; sif = SPICE_CONTAINEROF(sin->base.sif, SpiceCharDeviceInterface, base); /* protocol requires messages to be in network endianess */ |