Attachment 7866 Details for Bug 9147 – One more fix..

[patch] One more fix..

look (text/plain), 1.69 KB, created by Jeremy Allison on 2012-09-08 00:30:59 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Allison

Created: 2012-09-08 00:30:59 UTC

Size: 1.69 KB

patch

obsolete

>diff --git a/source3/lib/ldb_compat.c b/source3/lib/ldb_compat.c
>index 36a29e6..34e37e7 100644
>--- a/source3/lib/ldb_compat.c
>+++ b/source3/lib/ldb_compat.c
>@@ -99,7 +99,8 @@ char *ldb_binary_encode(void *mem_ctx, struct ldb_val val)
> 	unsigned char *buf = val.data;
> 
> 	for (i=0;i<val.length;i++) {
>-		if (!isprint(buf[i]) || strchr(" *()\\&|!\"", buf[i])) {
>+		unsigned int cval = buf[i];
>+		if (cval < 0x20 || cval > 0x7E || strchr(" *()\\&|!\"", buf[i])) {
> 			len += 2;
> 		}
> 	}
>@@ -108,7 +109,8 @@ char *ldb_binary_encode(void *mem_ctx, struct ldb_val val)
> 
> 	len = 0;
> 	for (i=0;i<val.length;i++) {
>-		if (!isprint(buf[i]) || strchr(" *()\\&|!\"", buf[i])) {
>+		unsigned int cval = buf[i];
>+		if (cval < 0x20 || cval > 0x7E || strchr(" *()\\&|!\"", buf[i])) {
> 			snprintf(ret+len, 4, "\\%02X", buf[i]);
> 			len += 3;
> 		} else {
>diff --git a/source4/lib/ldb/common/ldb_parse.c b/source4/lib/ldb/common/ldb_parse.c
>index b4eabf8..2a48d99 100644
>--- a/source4/lib/ldb/common/ldb_parse.c
>+++ b/source4/lib/ldb/common/ldb_parse.c
>@@ -124,7 +124,8 @@ char *ldb_binary_encode(TALLOC_CTX *mem_ctx, struct ldb_val val)
> 	unsigned char *buf = val.data;
> 
> 	for (i=0;i<val.length;i++) {
>-		if (!isprint(buf[i]) || strchr(" *()\\&|!\"", buf[i])) {
>+		unsigned int cval = buf[i];
>+		if (cval < 0x20 || cval > 0x7E || strchr(" *()\\&|!\"", buf[i])) {
> 			len += 2;
> 		}
> 	}
>@@ -133,7 +134,8 @@ char *ldb_binary_encode(TALLOC_CTX *mem_ctx, struct ldb_val val)
> 
> 	len = 0;
> 	for (i=0;i<val.length;i++) {
>-		if (!isprint(buf[i]) || strchr(" *()\\&|!\"", buf[i])) {
>+		unsigned int cval = buf[i];
>+		if (cval < 0x20 || cval > 0x7E || strchr(" *()\\&|!\"", buf[i])) {
> 			snprintf(ret+len, 4, "\\%02X", buf[i]);
> 			len += 3;
> 		} else {

Actions: View

Attachments on bug 9147: 7864 | 7865 | 7866 | 7874 | 7876 | 7879 | 7880 | 7881 | 7882 | 7883 | 7884 | 7885