Module Name:    src
Committed By:   agc
Date:           Sun Dec  6 17:43:05 UTC 2009

Modified Files:
        src/crypto/external/bsd/netpgp/dist/src/lib: bufgap.c bufgap.h defs.h
            ssh2pgp.c

Log Message:
Fix up some lint.

Change the license on new files to a 2-clause one.


To generate a diff of this commit:
cvs rdiff -u -r1.1 -r1.2 src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.c \
    src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h \
    src/crypto/external/bsd/netpgp/dist/src/lib/defs.h
cvs rdiff -u -r1.2 -r1.3 \
    src/crypto/external/bsd/netpgp/dist/src/lib/ssh2pgp.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.c
diff -u src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.c:1.1 src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.c:1.2
--- src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.c:1.1	Sat Dec  5 07:08:18 2009
+++ src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.c	Sun Dec  6 17:43:05 2009
@@ -1,7 +1,11 @@
-/* $NetBSD: bufgap.c,v 1.1 2009/12/05 07:08:18 agc Exp $ */
+/* $NetBSD: bufgap.c,v 1.2 2009/12/06 17:43:05 agc Exp $ */
 
-/*
- * Copyright © 1996-2009 Alistair Crooks.  All rights reserved.
+/*-
+ * Copyright (c) 1996-2009 The NetBSD Foundation, Inc.
+ * All rights reserved.
+ *
+ * This code is derived from software contributed to The NetBSD Foundation
+ * by Alistair Crooks ([email protected])
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -11,24 +15,18 @@
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *	This product includes software developed by Alistair G. Crooks.
- * 4. The name of the author may not be used to endorse or promote
- *    products derived from this software without specific prior written
- *    permission.
  *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS
- * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
- * GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
- * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
- * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
- * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
  */
 #include "config.h"
 
@@ -57,8 +55,8 @@
 #include "defs.h"
 
 /* macros to get subscripts in buffer */
-#define AFTSUB(bp, n)	((bp)->buf[n])
-#define BEFSUB(bp, n)	((bp)->buf[(bp)->size - (n) - 1])
+#define AFTSUB(bp, n)	((bp)->buf[(int)n])
+#define BEFSUB(bp, n)	((bp)->buf[(int)((bp)->size - (n) - 1)])
 
 /* initial allocation size */
 #ifndef CHUNKSIZE
@@ -108,8 +106,11 @@
 {
 	char	*cp;
 
+	if (n < 0) {
+		n = strlen(s);
+	}
 	NEWARRAY(char, cp, n + 1, "strnsave", return NULL);
-	(void) memcpy(cp, s, n);
+	(void) memcpy(cp, s, (size_t)n);
 	cp[n] = 0x0;
 	return cp;
 }
@@ -136,14 +137,14 @@
 		bp->size = (int) ((s.st_size / BGCHUNKSIZE) + 1) * BGCHUNKSIZE;
 		NEWARRAY(char, bp->buf, bp->size, "f_open", return 0);
 		cc = fread(&BEFSUB(bp, s.st_size), sizeof(char),
-							s.st_size, filep);
+						(size_t)s.st_size, filep);
 		(void) fclose(filep);
 		if (cc != s.st_size) {
 			FREE(bp->buf);
 			FREE(bp);
 			return 0;
 		}
-		bp->name = strnsave(__UNCONST(f), utfbytes(__UNCONST(f)));
+		bp->name = strnsave(__UNCONST(f), (int)utfbytes(__UNCONST(f)));
 		bp->bbc = s.st_size;
 		cp = &BEFSUB(bp, cc);
 		for (;;) {
@@ -153,7 +154,7 @@
 			bp->blc++;
 			cp++;
 		}
-		bp->bcc = utfnlen(&BEFSUB(bp, cc), cc);
+		bp->bcc = utfnlen(&BEFSUB(bp, cc), (size_t)cc);
 	}
 	return 1;
 }
@@ -180,7 +181,9 @@
 				if (rlen == 1) {
 					AFTSUB(bp, bp->abc) = BEFSUB(bp, bp->bbc);
 				} else {
-					(void) memmove(&AFTSUB(bp, bp->abc), &BEFSUB(bp, bp->bbc), rlen);
+					(void) memmove(&AFTSUB(bp, bp->abc),
+							&BEFSUB(bp, bp->bbc),
+							(size_t)rlen);
 				}
 				bp->acc++;
 				bp->bcc--;
@@ -201,7 +204,9 @@
 				if (rlen == 1) {
 					AFTSUB(bp, bp->abc) = BEFSUB(bp, bp->bbc);
 				} else {
-					(void) memmove(&AFTSUB(bp, bp->abc), &BEFSUB(bp, bp->bbc), rlen);
+					(void) memmove(&AFTSUB(bp, bp->abc),
+							&BEFSUB(bp, bp->bbc),
+							(size_t)rlen);
 				}
 				bp->acc++;
 				bp->bcc--;
@@ -237,7 +242,9 @@
 				if (rlen == 1) {
 					BEFSUB(bp, bp->bbc) = AFTSUB(bp, bp->abc);
 				} else {
-					(void) memmove(&BEFSUB(bp, bp->bbc), &AFTSUB(bp, bp->abc), rlen);
+					(void) memmove(&BEFSUB(bp, bp->bbc),
+							&AFTSUB(bp, bp->abc),
+							(size_t)rlen);
 				}
 				if (r == '\n') {
 					bp->blc++;
@@ -258,7 +265,9 @@
 				if (rlen == 1) {
 					BEFSUB(bp, bp->bbc) = AFTSUB(bp, bp->abc);
 				} else {
-					(void) memmove(&BEFSUB(bp, bp->bbc), &AFTSUB(bp, bp->abc), rlen);
+					(void) memmove(&BEFSUB(bp, bp->bbc),
+							&AFTSUB(bp, bp->abc),
+							(size_t)rlen);
 				}
 				if (r == '\n') {
 					bp->blc++;
@@ -294,9 +303,9 @@
 			}
 			return 1;
 		case BGFromHere:
-			return bufgap_seek(bp, bp->alc + off, BGFromBOF, BGLine);
+			return bufgap_seek(bp, (int64_t)(bp->alc + off), BGFromBOF, BGLine);
 		case BGFromEOF:
-			return bufgap_seek(bp, bp->alc + bp->blc + off, BGFromBOF, BGLine);
+			return bufgap_seek(bp, (int64_t)(bp->alc + bp->blc + off), BGFromBOF, BGLine);
 		}
 		break;
 	case BGChar:
@@ -312,9 +321,9 @@
 			}
 			return 1;
 		case BGFromHere:
-			return bufgap_seek(bp, bp->acc + off, BGFromBOF, BGChar);
+			return bufgap_seek(bp, (int64_t)(bp->acc + off), BGFromBOF, BGChar);
 		case BGFromEOF:
-			return bufgap_seek(bp, bp->acc + bp->bcc + off, BGFromBOF, BGChar);
+			return bufgap_seek(bp, (int64_t)(bp->acc + bp->bcc + off), BGFromBOF, BGChar);
 		}
 		break;
 	case BGByte:
@@ -330,9 +339,9 @@
 			}
 			return 1;
 		case BGFromHere:
-			return bufgap_seek(bp, bp->abc + off, BGFromBOF, BGByte);
+			return bufgap_seek(bp, (int64_t)(bp->abc + off), BGFromBOF, BGByte);
 		case BGFromEOF:
-			return bufgap_seek(bp, bp->abc + bp->bbc + off, BGFromBOF, BGByte);
+			return bufgap_seek(bp, (int64_t)(bp->abc + bp->bbc + off), BGFromBOF, BGByte);
 		}
 		break;
 	}
@@ -352,7 +361,7 @@
 {
 	int	cc;
 
-	cc = (bp->bcc < len) ? bp->bcc : len;
+	cc = (bp->bcc < len) ? (int)bp->bcc : (int)len;
 	(void) memcpy(dst, &BEFSUB(bp, bp->bbc), len);
 	return cc;
 }
@@ -407,7 +416,7 @@
 		if ((rlen = chartorune(&r, __UNCONST(s))) == 1) {
 			AFTSUB(bp, bp->abc) = *s;
 		} else {
-			(void) memmove(&AFTSUB(bp, bp->abc), s, rlen);
+			(void) memmove(&AFTSUB(bp, bp->abc), s, (size_t)rlen);
 		}
 		if (r == '\n') {
 			bp->alc++;
@@ -473,9 +482,9 @@
 	NEWARRAY(char, text, (to - from + 1), "bufgap_gettext", return NULL);
 	(void) bufgap_seek(bp, from, BGFromBOF, BGChar);
 	for (n = 0 ; n < to - from ; n++) {
-		text[n] = BEFSUB(bp, bp->bbc - n);
+		text[(int)n] = BEFSUB(bp, bp->bbc - n);
 	}
-	text[n] = 0x0;
+	text[(int)n] = 0x0;
 	(void) bufgap_seek(bp, off, BGFromBOF, BGChar);
 	return text;
 }
@@ -484,10 +493,10 @@
 int
 bufgap_write(bufgap_t *bp, FILE *filep)
 {
-	if (fwrite(bp->buf, sizeof(char), bp->abc, filep) != bp->abc) {
+	if (fwrite(bp->buf, sizeof(char), (size_t)bp->abc, filep) != (size_t)bp->abc) {
 		return 0;
 	}
-	if (fwrite(&BEFSUB(bp, bp->bbc), sizeof(char), bp->bbc, filep) != bp->bbc) {
+	if (fwrite(&BEFSUB(bp, bp->bbc), sizeof(char), (size_t)bp->bbc, filep) != (size_t)bp->bbc) {
 		return 0;
 	}
 	return 1;
Index: src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h
diff -u src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h:1.1 src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h:1.2
--- src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h:1.1	Sat Dec  5 07:08:18 2009
+++ src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h	Sun Dec  6 17:43:05 2009
@@ -1,7 +1,11 @@
-/* $Header: /cvsroot/src/crypto/external/bsd/netpgp/dist/src/lib/bufgap.h,v 1.1 2009/12/05 07:08:18 agc Exp $ */
+/* $NetBSD: bufgap.h,v 1.2 2009/12/06 17:43:05 agc Exp $ */
 
-/*
- * Copyright © 1996-1997 Alistair G. Crooks.  All rights reserved.
+/*-
+ * Copyright (c) 1996-2009 The NetBSD Foundation, Inc.
+ * All rights reserved.
+ *
+ * This code is derived from software contributed to The NetBSD Foundation
+ * by Alistair Crooks ([email protected])
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -11,24 +15,18 @@
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
  *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *	This product includes software developed by Alistair G. Crooks.
- * 4. The name of the author may not be used to endorse or promote
- *    products derived from this software without specific prior written
- *    permission.
  *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS
- * OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY
- * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
- * GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
- * WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
- * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
- * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
  */
 #ifndef BUFGAP_H_
 #define BUFGAP_H_ 20091023
Index: src/crypto/external/bsd/netpgp/dist/src/lib/defs.h
diff -u src/crypto/external/bsd/netpgp/dist/src/lib/defs.h:1.1 src/crypto/external/bsd/netpgp/dist/src/lib/defs.h:1.2
--- src/crypto/external/bsd/netpgp/dist/src/lib/defs.h:1.1	Sat Dec  5 07:08:18 2009
+++ src/crypto/external/bsd/netpgp/dist/src/lib/defs.h	Sun Dec  6 17:43:05 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: defs.h,v 1.1 2009/12/05 07:08:18 agc Exp $ */
+/* $NetBSD: defs.h,v 1.2 2009/12/06 17:43:05 agc Exp $ */
 
 /*-
  * Copyright (c) 2009 The NetBSD Foundation, Inc.
@@ -47,7 +47,7 @@
 #include <string.h>
 
 #define NEWARRAY(type,ptr,size,where,action) do {			\
-	if ((ptr = (type *) calloc(sizeof(type), (unsigned)(size))) == NULL) { \
+	if ((ptr = calloc(sizeof(type), (unsigned)(size))) == NULL) {	\
 		(void) fprintf(stderr, "%s: can't allocate %lu bytes\n", \
 			where, (unsigned long)(size * sizeof(type)));	\
 		action;							\
@@ -56,7 +56,8 @@
 
 #define RENEW(type,ptr,size,where,action) do {				\
 	type *_newptr;							\
-	if ((_newptr = (type *) realloc(ptr, sizeof(type) * (size))) == NULL) { \
+	_newptr = realloc(ptr, (size_t)(sizeof(type) * (size)));	\
+	if (_newptr == NULL) {						\
 		(void) fprintf(stderr, "%s: can't realloc %lu bytes\n",	\
 			where, (unsigned long)(size * sizeof(type)));	\
 		action;							\

Index: src/crypto/external/bsd/netpgp/dist/src/lib/ssh2pgp.c
diff -u src/crypto/external/bsd/netpgp/dist/src/lib/ssh2pgp.c:1.2 src/crypto/external/bsd/netpgp/dist/src/lib/ssh2pgp.c:1.3
--- src/crypto/external/bsd/netpgp/dist/src/lib/ssh2pgp.c:1.2	Sat Dec  5 17:23:39 2009
+++ src/crypto/external/bsd/netpgp/dist/src/lib/ssh2pgp.c	Sun Dec  6 17:43:05 2009
@@ -123,9 +123,11 @@
 			}
 		}
 		if (gotc) {
-			out[0] = (uint8_t) (in[0] << 2 | in[1] >> 4);
-			out[1] = (uint8_t) (in[1] << 4 | in[2] >> 2);
-			out[2] = (uint8_t) (((in[2] << 6) & 0xc0) | in[3]);
+			out[0] = (uint8_t)((unsigned)in[0] << 2 |
+						(unsigned)in[1] >> 4);
+			out[1] = (uint8_t)((unsigned)in[1] << 4 |
+						(unsigned)in[2] >> 2);
+			out[2] = (uint8_t)(((in[2] << 6) & 0xc0) | in[3]);
 			for (i = 0; i < gotc - 1; i++) {
 				*dst++ = out[i];
 			}
@@ -176,9 +178,9 @@
 	len = ntohl(len);
 	(void) bufgap_seek(bg, sizeof(len), BGFromHere, BGByte);
 	(void) bufgap_getbin(bg, buf, len);
-	bignum = BN_bin2bn((const unsigned char *)buf, len, NULL);
+	bignum = BN_bin2bn((const unsigned char *)buf, (int)len, NULL);
 	if (__ops_get_debug_level(__FILE__)) {
-		show(header, buf, len);
+		show(header, buf, (int)len);
 	}
 	(void) bufgap_seek(bg, len, BGFromHere, BGByte);
 	return bignum;
@@ -247,7 +249,7 @@
 	off = bufgap_tell(&bg, BGFromBOF, BGByte);
 
 	/* convert from base64 to binary */
-	cc = bufgap_getbin(&bg, buf, st.st_size);
+	cc = bufgap_getbin(&bg, buf, (size_t)st.st_size);
 	if ((space = strchr(buf, ' ')) != NULL) {
 		cc = (int)(space - buf);
 	}
@@ -258,7 +260,7 @@
 	if (__ops_get_debug_level(__FILE__)) {
 		show("decoded base64:", bin, cc);
 	}
-	bufgap_delete(&bg, bufgap_tell(&bg, BGFromEOF, BGByte));
+	bufgap_delete(&bg, (uint64_t)bufgap_tell(&bg, BGFromEOF, BGByte));
 	bufgap_insert(&bg, bin, cc);
 	bufgap_seek(&bg, off, BGFromBOF, BGByte);
 

Reply via email to