samba.git/lib/util/charset, branch master

lib: Fix blank line endings

2014-12-09T03:12:07+00:00

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison

lib: Fix a typo

2014-12-09T03:12:07+00:00

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison

lib: Simplify load_case_tables_library()

2014-12-07T23:33:06+00:00

We don't really need a talloc context here

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison

lib/util/tests: avoid some compiler warnings

2014-11-14T22:27:04+00:00

Signed-off-by: Stefan Metzmacher 
Reviewed-by: Jeremy Allison

charset: Avoid compiler warnings with --enable-developer

2014-09-19T16:11:11+00:00

These:

  [ 44/309] Compiling ../lib/util/charset/util_unistr_w.c
  ../../lib/util/charset/util_unistr_w.c: In function ‘strlower_w’:
  ../../lib/util/charset/util_unistr_w.c:182:127: warning: right-hand operand of comma expression has no effect [-Wunused-value]
      COPY_UCS2_CHAR(s,&v);

Signed-off-by: Martin Schwenke 
Reviewed-by: Volker Lendecke

lib: strings: Simplify strcasecmp

2014-08-05T23:05:14+00:00

This makes us fallback to strcasecmp early if any INVALID_CODEPOINT
appears. Without this patch we just continue to compare if both strings
happen to have an INVALID_CODEPOINT in the same spot.

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison

s4: tests: Added local.charset test for Bug 10716 - smbd constantly crashes when filename contains non-ascii character

2014-08-05T23:05:14+00:00

https://bugzilla.samba.org/show_bug.cgi?id=10716

Signed-off-by: Jeremy Allison

lib: strings: Fix the behavior of strncasecmp_m_handle() in the face of bad conversions.

2014-08-05T23:05:14+00:00

When either string has a bad conversion, we fall back to
doing raw ascii byte comparisons using strcasecmp(). This
is wrong - we should fall back to strncasecmp.

The problem is we've already stepped past the character
that failed the conversion, so we're not re-testing those
characters for comparison. This can have the effect of
causing strncasecmp_m_handle() to report that two strings
are identical when they are not, if the failed conversion
takes place at the end of the string.

The correct behavior is to step back to the point of
the string(s) that failed the conversion, and continue
the test from there.

This is a litle trickier than the previous fix, as
it requires converting the incoming n variable from
remaining characters to compare to remaining bytes to
compare.

As bytes are always the smallest character size
(1 byte) then it's safe to convert the remaining
characters to check by decrementing the source string
by the last character length (in bytes) and incrementing
the remaining bytes to scan by the same value, then
calling strncasecmp() with the stepped back strings
remaining.

Signed-off-by: Jeremy Allison

lib: strings: Fix the behavior of strcasecmp_m_handle() in the face of bad conversions.

2014-08-05T23:05:14+00:00

When either string has a bad conversion, we fall back to
doing raw ascii byte comparisons using strcasecmp().

The problem is we've already stepped past the character
that failed the conversion, so we're not re-testing those
characters for comparison. This can have the effect of
causing strcasecmp_m_handle() to report that two strings
are identical when they are not, if the failed conversion
takes place at the end of the string.

The correct behavior is to step back to the point of
the string(s) that failed the conversion, and continue
the test from there.

Found by  when investigating bug
10716 - smbd constantly crashes when filename contains non-ascii character.

Given the normal character set of utf-8, and an on
disk filename of ISO-8859-1 of file-é on disk hex
value: 66 69 6c 65 2d e9, an incoming open given the
correct utf8 name of file-é will collide when it
should not.

Fixes:

Bug 10716 - smbd constantly crashes when filename contains non-ascii character

https://bugzilla.samba.org/show_bug.cgi?id=10716

Signed-off-by: Jeremy Allison

lib/util/tests: add missing #include "torture/local/proto.h"

2014-04-02T07:03:44+00:00

Signed-off-by: Stefan Metzmacher 
Reviewed-by: Andrew Bartlett