public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Andreas Gruenbacher <agruen@suse.de>
To: acl-devel@nongnu.org
Cc: Anoop Vijayan <acv@linux.vnet.ibm.com>, xfs@oss.sgi.com
Subject: Re: [Acl-devel] Re: getfacl gives octal output for Hebrew user/group names
Date: Tue, 23 Jun 2009 11:48:01 +0200	[thread overview]
Message-ID: <200906231148.02088.agruen@suse.de> (raw)
In-Reply-To: <4A3F3304.1080901@linux.vnet.ibm.com>

On Monday 22 June 2009 09:30:12 Anoop Vijayan wrote:
> Also, acl_entry_to_any_str() in libacl/__acl_to_any_text.c calls quote.
> I could not find the corresponding unquote calls for these.

See acl_from_text() and several functions in setfacl/parse.c.

> If I wrap the above quote() calls with unquote(), the problem seems to be
> resolved. Attaching a patch for the same.

Yes, but this only proves that the quote and unquote functions work as 
intended, it doesn't help fix the underlying problem at all that special 
characters in file, user, or group names can mess up the output.

I have updated the quoting code in the acl and attr packages now so that it 
will only quote a few special characters line newline, carriage return, and a 
few delimiter characters. So this should work for Hebrew or other special 
characters now.

Note that there is no way to not quote backslash as backslash is the escape 
character used, so Samba names of the form "domain\user" will still be quoted. 
It's possible to teach winbindd to use a different character; otherwise, 
you'll have to live with the quoting.)

Thanks,
Andreas

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

      reply	other threads:[~2009-06-23  9:48 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-17 13:31 getfacl gives octal output for Hebrew user/group names Anoop Vijayan
2009-06-17 14:18 ` Eric Sandeen
2009-06-19 17:17 ` Andreas Gruenbacher
2009-06-22  7:30   ` Anoop Vijayan
2009-06-23  9:48     ` Andreas Gruenbacher [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200906231148.02088.agruen@suse.de \
    --to=agruen@suse.de \
    --cc=acl-devel@nongnu.org \
    --cc=acv@linux.vnet.ibm.com \
    --cc=xfs@oss.sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox