diff options
| author | guenther <> | 2014-04-19 11:46:39 +0000 |
|---|---|---|
| committer | guenther <> | 2014-04-19 11:46:39 +0000 |
| commit | f376030ff7fe24f4ad44a56c2d8ba8796c9ab3f0 (patch) | |
| tree | 1c4f9b095801061d57e575e80f245cef8a59a27f /src/lib/libcrypto/asn1 | |
| parent | 597610ccf04bbca4e88fac7988877d0a06e02211 (diff) | |
| download | openbsd-f376030ff7fe24f4ad44a56c2d8ba8796c9ab3f0.tar.gz openbsd-f376030ff7fe24f4ad44a56c2d8ba8796c9ab3f0.tar.bz2 openbsd-f376030ff7fe24f4ad44a56c2d8ba8796c9ab3f0.zip | |
We'll interpret a (void) cast on snprintf() to mean it's been verified that
truncation is either desirable, not an issue, or is detected and handled later
ok deraadt@
Diffstat (limited to 'src/lib/libcrypto/asn1')
| -rw-r--r-- | src/lib/libcrypto/asn1/a_gentm.c | 2 | ||||
| -rw-r--r-- | src/lib/libcrypto/asn1/a_mbstr.c | 4 | ||||
| -rw-r--r-- | src/lib/libcrypto/asn1/a_strex.c | 6 | ||||
| -rw-r--r-- | src/lib/libcrypto/asn1/a_utctm.c | 2 | ||||
| -rw-r--r-- | src/lib/libcrypto/asn1/asn1_lib.c | 4 | ||||
| -rw-r--r-- | src/lib/libcrypto/asn1/asn1_par.c | 8 |
6 files changed, 13 insertions, 13 deletions
diff --git a/src/lib/libcrypto/asn1/a_gentm.c b/src/lib/libcrypto/asn1/a_gentm.c index 856aaf0c77..f331bff320 100644 --- a/src/lib/libcrypto/asn1/a_gentm.c +++ b/src/lib/libcrypto/asn1/a_gentm.c | |||
| @@ -244,7 +244,7 @@ ASN1_GENERALIZEDTIME_adj(ASN1_GENERALIZEDTIME *s, time_t t, int offset_day, | |||
| 244 | s->data = (unsigned char *)p; | 244 | s->data = (unsigned char *)p; |
| 245 | } | 245 | } |
| 246 | 246 | ||
| 247 | (void) snprintf(p, len, "%04d%02d%02d%02d%02d%02dZ", ts->tm_year + 1900, | 247 | snprintf(p, len, "%04d%02d%02d%02d%02d%02dZ", ts->tm_year + 1900, |
| 248 | ts->tm_mon + 1, ts->tm_mday, ts->tm_hour, ts->tm_min, ts->tm_sec); | 248 | ts->tm_mon + 1, ts->tm_mday, ts->tm_hour, ts->tm_min, ts->tm_sec); |
| 249 | s->length = strlen(p); | 249 | s->length = strlen(p); |
| 250 | s->type = V_ASN1_GENERALIZEDTIME; | 250 | s->type = V_ASN1_GENERALIZEDTIME; |
diff --git a/src/lib/libcrypto/asn1/a_mbstr.c b/src/lib/libcrypto/asn1/a_mbstr.c index 5a909d6ae2..6528161c41 100644 --- a/src/lib/libcrypto/asn1/a_mbstr.c +++ b/src/lib/libcrypto/asn1/a_mbstr.c | |||
| @@ -150,14 +150,14 @@ ASN1_mbstring_ncopy(ASN1_STRING **out, const unsigned char *in, int len, | |||
| 150 | 150 | ||
| 151 | if ((minsize > 0) && (nchar < minsize)) { | 151 | if ((minsize > 0) && (nchar < minsize)) { |
| 152 | ASN1err(ASN1_F_ASN1_MBSTRING_NCOPY, ASN1_R_STRING_TOO_SHORT); | 152 | ASN1err(ASN1_F_ASN1_MBSTRING_NCOPY, ASN1_R_STRING_TOO_SHORT); |
| 153 | (void) snprintf(strbuf, sizeof strbuf, "%ld", minsize); | 153 | snprintf(strbuf, sizeof strbuf, "%ld", minsize); |
| 154 | ERR_add_error_data(2, "minsize=", strbuf); | 154 | ERR_add_error_data(2, "minsize=", strbuf); |
| 155 | return -1; | 155 | return -1; |
| 156 | } | 156 | } |
| 157 | 157 | ||
| 158 | if ((maxsize > 0) && (nchar > maxsize)) { | 158 | if ((maxsize > 0) && (nchar > maxsize)) { |
| 159 | ASN1err(ASN1_F_ASN1_MBSTRING_NCOPY, ASN1_R_STRING_TOO_LONG); | 159 | ASN1err(ASN1_F_ASN1_MBSTRING_NCOPY, ASN1_R_STRING_TOO_LONG); |
| 160 | (void) snprintf(strbuf, sizeof strbuf, "%ld", maxsize); | 160 | snprintf(strbuf, sizeof strbuf, "%ld", maxsize); |
| 161 | ERR_add_error_data(2, "maxsize=", strbuf); | 161 | ERR_add_error_data(2, "maxsize=", strbuf); |
| 162 | return -1; | 162 | return -1; |
| 163 | } | 163 | } |
diff --git a/src/lib/libcrypto/asn1/a_strex.c b/src/lib/libcrypto/asn1/a_strex.c index 7dc531a2d8..e92c166303 100644 --- a/src/lib/libcrypto/asn1/a_strex.c +++ b/src/lib/libcrypto/asn1/a_strex.c | |||
| @@ -125,12 +125,12 @@ static int do_esc_char(unsigned long c, unsigned char flags, char *do_quotes, ch | |||
| 125 | if(c > 0xffffffffL) | 125 | if(c > 0xffffffffL) |
| 126 | return -1; | 126 | return -1; |
| 127 | if(c > 0xffff) { | 127 | if(c > 0xffff) { |
| 128 | (void) snprintf(tmphex, sizeof tmphex, "\\W%08lX", c); | 128 | snprintf(tmphex, sizeof tmphex, "\\W%08lX", c); |
| 129 | if(!io_ch(arg, tmphex, 10)) return -1; | 129 | if(!io_ch(arg, tmphex, 10)) return -1; |
| 130 | return 10; | 130 | return 10; |
| 131 | } | 131 | } |
| 132 | if(c > 0xff) { | 132 | if(c > 0xff) { |
| 133 | (void) snprintf(tmphex, sizeof tmphex, "\\U%04lX", c); | 133 | snprintf(tmphex, sizeof tmphex, "\\U%04lX", c); |
| 134 | if(!io_ch(arg, tmphex, 6)) return -1; | 134 | if(!io_ch(arg, tmphex, 6)) return -1; |
| 135 | return 6; | 135 | return 6; |
| 136 | } | 136 | } |
| @@ -149,7 +149,7 @@ static int do_esc_char(unsigned long c, unsigned char flags, char *do_quotes, ch | |||
| 149 | return 2; | 149 | return 2; |
| 150 | } | 150 | } |
| 151 | if(chflgs & (ASN1_STRFLGS_ESC_CTRL|ASN1_STRFLGS_ESC_MSB)) { | 151 | if(chflgs & (ASN1_STRFLGS_ESC_CTRL|ASN1_STRFLGS_ESC_MSB)) { |
| 152 | (void) snprintf(tmphex, sizeof tmphex, "\\%02X", chtmp); | 152 | snprintf(tmphex, sizeof tmphex, "\\%02X", chtmp); |
| 153 | if(!io_ch(arg, tmphex, 3)) return -1; | 153 | if(!io_ch(arg, tmphex, 3)) return -1; |
| 154 | return 3; | 154 | return 3; |
| 155 | } | 155 | } |
diff --git a/src/lib/libcrypto/asn1/a_utctm.c b/src/lib/libcrypto/asn1/a_utctm.c index a2325f2525..0120b952b3 100644 --- a/src/lib/libcrypto/asn1/a_utctm.c +++ b/src/lib/libcrypto/asn1/a_utctm.c | |||
| @@ -205,7 +205,7 @@ ASN1_UTCTIME *ASN1_UTCTIME_adj(ASN1_UTCTIME *s, time_t t, | |||
| 205 | s->data=(unsigned char *)p; | 205 | s->data=(unsigned char *)p; |
| 206 | } | 206 | } |
| 207 | 207 | ||
| 208 | (void) snprintf(p,len,"%02d%02d%02d%02d%02d%02dZ",ts->tm_year%100, | 208 | snprintf(p,len,"%02d%02d%02d%02d%02d%02dZ",ts->tm_year%100, |
| 209 | ts->tm_mon+1,ts->tm_mday,ts->tm_hour,ts->tm_min,ts->tm_sec); | 209 | ts->tm_mon+1,ts->tm_mday,ts->tm_hour,ts->tm_min,ts->tm_sec); |
| 210 | s->length=strlen(p); | 210 | s->length=strlen(p); |
| 211 | s->type=V_ASN1_UTCTIME; | 211 | s->type=V_ASN1_UTCTIME; |
diff --git a/src/lib/libcrypto/asn1/asn1_lib.c b/src/lib/libcrypto/asn1/asn1_lib.c index 49d650b125..86cfdd3967 100644 --- a/src/lib/libcrypto/asn1/asn1_lib.c +++ b/src/lib/libcrypto/asn1/asn1_lib.c | |||
| @@ -467,8 +467,8 @@ asn1_add_error(const unsigned char *address, int offset) | |||
| 467 | { | 467 | { |
| 468 | char buf1[DECIMAL_SIZE(address) + 1], buf2[DECIMAL_SIZE(offset) + 1]; | 468 | char buf1[DECIMAL_SIZE(address) + 1], buf2[DECIMAL_SIZE(offset) + 1]; |
| 469 | 469 | ||
| 470 | (void) snprintf(buf1, sizeof buf1, "%lu", (unsigned long)address); | 470 | snprintf(buf1, sizeof buf1, "%lu", (unsigned long)address); |
| 471 | (void) snprintf(buf2, sizeof buf2, "%d", offset); | 471 | snprintf(buf2, sizeof buf2, "%d", offset); |
| 472 | ERR_add_error_data(4, "address=", buf1, " offset=", buf2); | 472 | ERR_add_error_data(4, "address=", buf1, " offset=", buf2); |
| 473 | } | 473 | } |
| 474 | 474 | ||
diff --git a/src/lib/libcrypto/asn1/asn1_par.c b/src/lib/libcrypto/asn1/asn1_par.c index 064b3dcb22..8085d1e570 100644 --- a/src/lib/libcrypto/asn1/asn1_par.c +++ b/src/lib/libcrypto/asn1/asn1_par.c | |||
| @@ -85,13 +85,13 @@ asn1_print_info(BIO *bp, int tag, int xclass, int constructed, | |||
| 85 | 85 | ||
| 86 | p = str; | 86 | p = str; |
| 87 | if ((xclass & V_ASN1_PRIVATE) == V_ASN1_PRIVATE) | 87 | if ((xclass & V_ASN1_PRIVATE) == V_ASN1_PRIVATE) |
| 88 | (void) snprintf(str, sizeof str, "priv [ %d ] ", tag); | 88 | snprintf(str, sizeof str, "priv [ %d ] ", tag); |
| 89 | else if ((xclass & V_ASN1_CONTEXT_SPECIFIC) == V_ASN1_CONTEXT_SPECIFIC) | 89 | else if ((xclass & V_ASN1_CONTEXT_SPECIFIC) == V_ASN1_CONTEXT_SPECIFIC) |
| 90 | (void) snprintf(str, sizeof str, "cont [ %d ]", tag); | 90 | snprintf(str, sizeof str, "cont [ %d ]", tag); |
| 91 | else if ((xclass & V_ASN1_APPLICATION) == V_ASN1_APPLICATION) | 91 | else if ((xclass & V_ASN1_APPLICATION) == V_ASN1_APPLICATION) |
| 92 | (void) snprintf(str, sizeof str, "appl [ %d ]", tag); | 92 | snprintf(str, sizeof str, "appl [ %d ]", tag); |
| 93 | else if (tag > 30) | 93 | else if (tag > 30) |
| 94 | (void) snprintf(str, sizeof str, "<ASN1 %d>", tag); | 94 | snprintf(str, sizeof str, "<ASN1 %d>", tag); |
| 95 | else | 95 | else |
| 96 | p = ASN1_tag2str(tag); | 96 | p = ASN1_tag2str(tag); |
| 97 | 97 | ||
