All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Brook <paul@codesourcery.com>
To: qemu-devel@nongnu.org
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>
Subject: Re: [Qemu-devel] Use of PATH_MAX
Date: Fri, 16 May 2008 15:02:10 +0100	[thread overview]
Message-ID: <200805161502.10631.paul@codesourcery.com> (raw)
In-Reply-To: <18477.36259.30442.280531@mariner.uk.xensource.com>

On Friday 16 May 2008, Ian Jackson wrote:
> There are a couple of places where we use PATH_MAX.  I don't think
> this is right.  PATH_MAX is a #define specified by POSIX, SuSv3 etc.
> But it isn't guaranteed to be defined or necessarily very useful.
>
> In particular, it may be defined to a very large value (larger than a
> practical static buffer).  Or on systems where the maximum pathname
> length varies (for example, it depends on the underlying filesystem)
> it may be not defined at all and applications which really need to
> know are supposed to use pathconf.
>
> I think it would be better to invent a new name for the maximum path
> length supported by qemu's statically-sized buffers.  This would
> replace both the uses of PATH_MAX (in block.c, linux-user/path.c, and
> block-vvfat.c) but also direct use of (eg) 1024 in many places.

Better would be to not have the static length buffers in the first place. 
That's the whole reason you're seeing this problem in the first place. 

And yes, I do have real situation where a 1024 character buffer isn't big 
enough.

Paul

  parent reply	other threads:[~2008-05-16 14:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-16 13:35 [Qemu-devel] Use of PATH_MAX Ian Jackson
2008-05-16 14:00 ` Anthony Liguori
2008-05-16 14:09   ` Warner Losh
2008-05-16 14:35     ` Anthony Liguori
2008-05-16 14:52       ` Ian Jackson
2008-05-16 14:02 ` Paul Brook [this message]
2008-05-16 14:21   ` Ian Jackson
2008-05-16 14:24     ` Ian Jackson
2008-05-16 14:32       ` Anthony Liguori
2008-05-16 14:46         ` Ian Jackson

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=200805161502.10631.paul@codesourcery.com \
    --to=paul@codesourcery.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=qemu-devel@nongnu.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.