emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Pascal Brisset writes:

Browse Source 
uuencode fails to encode binary data because it right-shifts
bytes as signed chars and keeps the duplicated sign bits.

The original base64_encode() from wget/http.c is broken as well,
but it is only used to encode ascii data.

-- Pascal
This commit is contained in:
Eric Andersen 2004-01-26 07:17:30 +00:00
parent 7018385fe7
commit f2ec37902a
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
coreutils/uuencode.c
View File

@ -59,7 +59,7 @@ static const char tbl_std[65] = {
* buffer of at least 1+BASE64_LENGTH(length) bytes.
* where BASE64_LENGTH(len) = (4 * ((LENGTH + 2) / 3))
*/
static void uuencode (const char *s, const char *store, const int length, const char *tbl)
static void uuencode (const unsigned char *s, const char *store, const int length, const char *tbl)
{
int i;
unsigned char *p = (unsigned char *)store;

4
networking/wget.c
View File

@ -119,7 +119,7 @@ static char *safe_fgets(char *s, int size, FILE *stream)
* oops... isn't something similar in uuencode.c?
* It would be better to use already existing code
*/
char *base64enc(char *p, char *buf, int len) {
char *base64enc(unsigned char *p, char *buf, int len) {
char al[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"
"0123456789+/";
@ -837,7 +837,7 @@ progressmeter(int flag)
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
* SUCH DAMAGE.
*
* $Id: wget.c,v 1.66 2004/01/17 23:07:14 bug1 Exp $
* $Id: wget.c,v 1.67 2004/01/26 07:17:30 andersen Exp $
*/