Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAK3gATc0uTjFeF5M_vJ-ia=o8s--V40HQbQqT4=SExsqeqr3cw@mail.gmail.com>
Date: Tue, 1 Sep 2026 12:56:52 +0100
From: Ricardo Barreira <rsf@...gle.com>
To: musl@...ts.openwall.com
Subject: Re: [PATCH] ldso: increase buffer size to fix loading error with long paths

Correction: I meant path_open and not path_open_library. The latter is
the name of the function on the Android repo:
https://android.googlesource.com/platform/external/musl/+/refs/heads/main/ldso/dynlink.c

Apologies for the confusion!

Kind regards,
Ricardo

On Tue, 1 Sept 2026 at 12:26, Ricardo Barreira <rsf@...gle.com> wrote:
>
> path_open_library fills this buffer with a path, so it needs to be
> sized accordingly with PATH_MAX+1.
>
> Signed-off-by: Ricardo Barreira <rsf@...gle.com>
> ---
>  ldso/dynlink.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/ldso/dynlink.c b/ldso/dynlink.c
> index 10471b23..32096f55 100644
> --- a/ldso/dynlink.c
> +++ b/ldso/dynlink.c
> @@ -1056,7 +1056,7 @@ static void makefuncdescs(struct dso *p)
>
>  static struct dso *load_library(const char *name, struct dso *needed_by)
>  {
> -       char buf[2*NAME_MAX+2];
> +       char buf[PATH_MAX+1];
>         const char *pathname;
>         unsigned char *map;
>         struct dso *p, temp_dso = {0};
> --
> 2.55.0.897.gb25b4bd76c-goog
>

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.