From patchwork Tue Jul 3 17:06:47 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Gabriel Krisman Bertazi X-Patchwork-Id: 938820 Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-ext4-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=collabora.co.uk Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 41KrCZ0DmNz9s3x for ; Wed, 4 Jul 2018 03:07:38 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934191AbeGCRHg (ORCPT ); Tue, 3 Jul 2018 13:07:36 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:33374 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934187AbeGCRHf (ORCPT ); Tue, 3 Jul 2018 13:07:35 -0400 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: krisman) with ESMTPSA id 6D2F72605C7 From: Gabriel Krisman Bertazi To: tytso@mit.edu Cc: linux-ext4@vger.kernel.org, darrick.wong@oracle.com, kernel@collabora.com, Gabriel Krisman Bertazi Subject: [PATCH 07/20] nls: Add new interface for string comparisons Date: Tue, 3 Jul 2018 13:06:47 -0400 Message-Id: <20180703170700.9306-8-krisman@collabora.co.uk> X-Mailer: git-send-email 2.18.0 In-Reply-To: <20180703170700.9306-1-krisman@collabora.co.uk> References: <20180703170700.9306-1-krisman@collabora.co.uk> Sender: linux-ext4-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org The existing stricmp() interface is limited by not accepting separated length parameters for each string being compared. This is a problem for charsets doing normalization or full casefold comparison, since different sized strings can still be matched. To resolve this problem, this patch implements a new interface, allowing charsets to do the comparison, if needed. The original stricmp is left in the code, while all callers are not converted, but was rewritten the new interface. Signed-off-by: Gabriel Krisman Bertazi --- include/linux/nls.h | 42 +++++++++++++++++++++++++++++++++++++++--- 1 file changed, 39 insertions(+), 3 deletions(-) diff --git a/include/linux/nls.h b/include/linux/nls.h index e422bd52afbb..d8b49f53c123 100644 --- a/include/linux/nls.h +++ b/include/linux/nls.h @@ -3,6 +3,7 @@ #define _LINUX_NLS_H #include +#include /* Unicode has changed over the years. Unicode code points no longer * fit into 16 bits; as of Unicode 5 valid code points range from 0 @@ -21,11 +22,18 @@ typedef u16 wchar_t; /* Arbitrary Unicode character */ typedef u32 unicode_t; +struct nls_table; struct nls_ops { int (*uni2char) (wchar_t uni, unsigned char *out, int boundlen); int (*char2uni) (const unsigned char *rawstring, int boundlen, wchar_t *uni); + int (*strncmp)(const struct nls_table *charset, + const unsigned char *str1, size_t len1, + const unsigned char *str2, size_t len2); + int (*strncasecmp)(const struct nls_table *charset, + const unsigned char *str1, size_t len1, + const unsigned char *str2, size_t len2); }; struct nls_table { @@ -106,10 +114,17 @@ static inline unsigned char nls_toupper(struct nls_table *t, unsigned char c) return nc ? nc : c; } -static inline int nls_strnicmp(struct nls_table *t, const unsigned char *s1, - const unsigned char *s2, int len) +static inline int nls_strncasecmp(struct nls_table *t, + const unsigned char *s1, size_t len1, + const unsigned char *s2, size_t len2) { - while (len--) { + if (t->ops->strncasecmp) + return t->ops->strncasecmp(t, s1, len1, s2, len2); + + if (len1 != len2) + return 1; + + while (len1--) { if (nls_tolower(t, *s1++) != nls_tolower(t, *s2++)) return 1; } @@ -117,6 +132,27 @@ static inline int nls_strnicmp(struct nls_table *t, const unsigned char *s1, return 0; } +static inline int nls_strncmp(struct nls_table *t, + const unsigned char *s1, size_t len1, + const unsigned char *s2, size_t len2) +{ + if (t->ops->strncmp) + return t->ops->strncmp(t, s1, len1, s2, len2); + + if (len1 != len2) + return 1; + + /* strnicmp did not return negative values. So let's keep the + * abi for now */ + return !!memcmp(s1, s2, len1); +} + +static inline int nls_strnicmp(struct nls_table *t, const unsigned char *s1, + const unsigned char *s2, int len) +{ + return nls_strncasecmp(t, s1, len, s2, len); +} + /* * nls_nullsize - return length of null character for codepage * @codepage - codepage for which to return length of NULL terminator