X-Git-Url: http://www.average.org/gitweb/?p=pam_pcsc_cr.git;a=blobdiff_plain;f=serial.c;h=8a24abd3db1d11f19bdce85a905420936e4a155f;hp=0719746a52eb7c001a6605c716e1de62df7c1d11;hb=9cfb078547dfafdb01cfd87c05acffdbc07255ee;hpb=ecefc4912f36f9153e63e7e0ee90d8ef69898fed

diff --git a/serial.c b/serial.c
index 0719746..8a24abd 100644
--- a/serial.c
+++ b/serial.c
@@ -8,6 +8,7 @@ int serial_init(serializer_t *srl, void *buffer, int size)
 {
 	srl->buffer = srl->cursor = buffer;
 	srl->bufsize = size;
+	return 0;
 }
 
 int serial_switch(serializer_t *srl, void *buffer, int size)
@@ -18,14 +19,16 @@ int serial_switch(serializer_t *srl, void *buffer, int size)
 	srl->buffer = buffer;
 	srl->bufsize = size;
 	srl->cursor = buffer + used;
+	return 0;
 }
 
-int serial_put(serializer_t *srl, void *item, int size)
+int serial_put(serializer_t *srl, const void *item, int size)
 {
 	int left = srl->bufsize - (srl->cursor - srl->buffer);
+
 	if (left < size + sizeof(short)) return left - sizeof(short);
 	*((short *)srl->cursor) = size;
-	srl->cursor += 2;
+	srl->cursor += sizeof(short);
 	if (size) memcpy(srl->cursor, item, size);
 	srl->cursor += size;
 	return size;
@@ -33,10 +36,13 @@ int serial_put(serializer_t *srl, void *item, int size)
 
 int serial_get(serializer_t *srl, void *item, int bufsize)
 {
+	int left = srl->bufsize - (srl->cursor - srl->buffer);
 	short isize = *((short *)srl->cursor);
-	if (isize > bufsize || isize == 0) return isize;
+
+	if (isize > bufsize) return isize;
+	if (isize + sizeof(short) > left) return -1;
 	srl->cursor += sizeof(short);
-	memcpy(item, srl->cursor, isize);
+	if (isize) memcpy(item, srl->cursor, isize);
 	srl->cursor += isize;
 	return isize;
 }