linux-ext4 - Re: [PATCH 5/5] libquota: remove NLS support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 4 Oct 2011 09:24:25 -0700
From:	Aditya Kali <adityakali@...gle.com>
To:	"Theodore Ts'o" <tytso@....edu>
Cc:	Ext4 Developers List <linux-ext4@...r.kernel.org>
Subject: Re: [PATCH 5/5] libquota: remove NLS support

Hi Ted,

Thanks for the patches.
I do have patches that removes log_err() and log_fatal() and all
'exit()' calls (and bunch of other cleanups). I will send those
patches as soon as possible.

Thanks,

On Tue, Oct 4, 2011 at 9:16 AM, Theodore Ts'o <tytso@....edu> wrote:
>
> In general libraries should never (a) call exit() or (b) print output
> directly to the stdout (they might be used by GUI programs.  From (b)
> follows (c), should never call internationalization functions
> directly.
>
> Also, since po/POTFILES.in wasn't edited, these strings weren't
> getting included in e2fsprogs.pot for translation, so the _()
> indirection didn't actually buy us anything.
>
> We eventually need to nuke all of the log_fatal() and log_err() from
> libquota, so best thing to do for now is remove NLS support
> completely; no point whipsawing the translators with strings to
> translate that will be disappearing soon anyway!
>
> Signed-off-by: "Theodore Ts'o" <tytso@....edu>
> ---
>  lib/quota/common.h |   30 ++++--------------------------
>  1 files changed, 4 insertions(+), 26 deletions(-)
>
> diff --git a/lib/quota/common.h b/lib/quota/common.h
> index 48f191f..7b88003 100644
> --- a/lib/quota/common.h
> +++ b/lib/quota/common.h
> @@ -13,41 +13,19 @@
>  # endif
>  #endif
>
> -#ifdef ENABLE_NLS
> -#include <libintl.h>
> -#include <locale.h>
> -#define _(a) (gettext (a))
> -#ifdef gettext_noop
> -#define N_(a) gettext_noop (a)
> -#else
> -#define N_(a) (a)
> -#endif
> -#define P_(singular, plural, n) (ngettext (singular, plural, n))
> -#ifndef NLS_CAT_NAME
> -#define NLS_CAT_NAME "e2fsprogs"
> -#endif
> -#ifndef LOCALEDIR
> -#define LOCALEDIR "/usr/share/locale"
> -#endif
> -#else
> -#define _(a) (a)
> -#define N_(a) a
> -#define P_(singular, plural, n) ((n) == 1 ? (singular) : (plural))
> -#endif
> -
>  #define log_fatal(exit_code, format, ...)      do { \
> -               fprintf(stderr, _("[FATAL] %s:%d:%s:: " format "\n"), \
> +               fprintf(stderr, "[FATAL] %s:%d:%s:: " format "\n", \
>                        __FILE__, __LINE__, __func__, __VA_ARGS__); \
>                exit(exit_code); \
>        } while (0)
>
>  #define log_err(format, ...)   fprintf(stderr, \
> -                               _("[ERROR] %s:%d:%s:: " format "\n"), \
> +                               "[ERROR] %s:%d:%s:: " format "\n", \
>                                __FILE__, __LINE__, __func__, __VA_ARGS__)
>
>  #ifdef DEBUG_QUOTA
>  # define log_debug(format, ...)        fprintf(stderr, \
> -                               _("[DEBUG] %s:%d:%s:: " format "\n"), \
> +                               "[DEBUG] %s:%d:%s:: " format "\n", \
>                                __FILE__, __LINE__, __func__, __VA_ARGS__)
>  #else
>  # define log_debug(format, ...)
> @@ -55,7 +33,7 @@
>
>  #define BUG_ON(x)              do { if ((x)) { \
>                                        fprintf(stderr, \
> -                                               _("BUG_ON: %s:%d:: ##x"), \
> +                                               "BUG_ON: %s:%d:: ##x", \
>                                                __FILE__, __LINE__); \
>                                        exit(2); \
>                                } } while (0)
> --
> 1.7.4.1.22.gec8e1.dirty
>



--
Aditya
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html