Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Unified Diff: third_party/modp_b64/modp_b64.cc

Issue 11688006: Make modp_b64 and base::base64 compile on Win64 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/modp_b64/modp_b64.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/modp_b64/modp_b64.cc
===================================================================
--- third_party/modp_b64/modp_b64.cc (revision 174668)
+++ third_party/modp_b64/modp_b64.cc (working copy)
@@ -77,20 +77,22 @@
#define CHARPAD '\0'
#endif
-int modp_b64_encode(char* dest, const char* str, int len)
+size_t modp_b64_encode(char* dest, const char* str, size_t len)
{
- int i;
+ size_t i = 0;
uint8_t* p = (uint8_t*) dest;
/* unsigned here is important! */
uint8_t t1, t2, t3;
- for (i = 0; i < len - 2; i += 3) {
- t1 = str[i]; t2 = str[i+1]; t3 = str[i+2];
- *p++ = e0[t1];
- *p++ = e1[((t1 & 0x03) << 4) | ((t2 >> 4) & 0x0F)];
- *p++ = e1[((t2 & 0x0F) << 2) | ((t3 >> 6) & 0x03)];
- *p++ = e2[t3];
+ if (len > 2) {
+ for (; i < len - 2; i += 3) {
+ t1 = str[i]; t2 = str[i+1]; t3 = str[i+2];
+ *p++ = e0[t1];
+ *p++ = e1[((t1 & 0x03) << 4) | ((t2 >> 4) & 0x0F)];
+ *p++ = e1[((t2 & 0x0F) << 2) | ((t3 >> 6) & 0x03)];
+ *p++ = e2[t3];
+ }
}
switch (len - i) {
@@ -124,7 +126,7 @@
/* if padding is used, then the message must be at least
4 chars and be a multiple of 4.
there can be at most 2 pad chars at the end */
- if (len < 4 || (len % 4 != 0)) return -1;
+ if (len < 4 || (len % 4 != 0)) return MODP_B64_ERROR;
if (src[len-1] == CHARPAD) {
len--;
if (src[len -1] == CHARPAD) {
@@ -133,9 +135,9 @@
}
#endif /* DOPAD */
- int i;
+ size_t i;
int leftover = len % 4;
- int chunks = (leftover == 0) ? len / 4 - 1 : len /4;
+ size_t chunks = (leftover == 0) ? len / 4 - 1 : len /4;
uint8_t* p = (uint8_t*) dest;
uint32_t x = 0;
@@ -146,7 +148,7 @@
x = d0[y >> 24 & 0xff] | d1[y >> 16 & 0xff] |
d2[y >> 8 & 0xff] | d3[y & 0xff];
- if (x >= BADCHAR) return -1;
+ if (x >= BADCHAR) return MODP_B64_ERROR;
*destInt = x << 8;
p += 3;
destInt = (uint32_t*)p;
@@ -157,7 +159,7 @@
case 0:
x = d0[y >> 24 & 0xff] | d1[y >> 16 & 0xff] |
d2[y >> 8 & 0xff] | d3[y & 0xff];
- if (x >= BADCHAR) return -1;
+ if (x >= BADCHAR) return MODP_B64_ERROR;
*p++ = ((uint8_t*)&x)[1];
*p++ = ((uint8_t*)&x)[2];
*p = ((uint8_t*)&x)[3];
@@ -178,13 +180,13 @@
break;
}
- if (x >= BADCHAR) return -1;
+ if (x >= BADCHAR) return MODP_B64_ERROR;
return 3*chunks + (6*leftover)/8;
}
#else /* LITTLE ENDIAN -- INTEL AND FRIENDS */
-int modp_b64_decode(char* dest, const char* src, int len)
+size_t modp_b64_decode(char* dest, const char* src, size_t len)
{
if (len == 0) return 0;
@@ -193,7 +195,7 @@
* if padding is used, then the message must be at least
* 4 chars and be a multiple of 4
*/
- if (len < 4 || (len % 4 != 0)) return -1; /* error */
+ if (len < 4 || (len % 4 != 0)) return MODP_B64_ERROR; /* error */
/* there can be at most 2 pad chars at the end */
if (src[len-1] == CHARPAD) {
len--;
@@ -203,9 +205,9 @@
}
#endif
- int i;
+ size_t i;
int leftover = len % 4;
- int chunks = (leftover == 0) ? len / 4 - 1 : len /4;
+ size_t chunks = (leftover == 0) ? len / 4 - 1 : len /4;
uint8_t* p = (uint8_t*)dest;
uint32_t x = 0;
@@ -218,7 +220,7 @@
d2[(y >> 16) & 0xff] |
d3[(y >> 24) & 0xff];
- if (x >= BADCHAR) return -1;
+ if (x >= BADCHAR) return MODP_B64_ERROR;
*destInt = x ;
p += 3;
destInt = (uint32_t*)p;
@@ -232,7 +234,7 @@
d2[(y >> 16) & 0xff] |
d3[(y >> 24) & 0xff];
- if (x >= BADCHAR) return -1;
+ if (x >= BADCHAR) return MODP_B64_ERROR;
*p++ = ((uint8_t*)(&x))[0];
*p++ = ((uint8_t*)(&x))[1];
*p = ((uint8_t*)(&x))[2];
@@ -255,7 +257,7 @@
break;
}
- if (x >= BADCHAR) return -1;
+ if (x >= BADCHAR) return MODP_B64_ERROR;
return 3*chunks + (6*leftover)/8;
}
« no previous file with comments | « third_party/modp_b64/modp_b64.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698