Re: [patch] Expand tildes in solib-search-path entries.

[Thiago Jung Bauermann <thiago.bauermann@linaro.org>]

Hello Pedro,

On Thu, 2012-05-24 at 16:30 +0100, Pedro Alves wrote:
> On 05/22/2012 07:17 PM, Thiago Jung Bauermann wrote:
> 
> > diff --git a/gdb/source.c b/gdb/source.c
> > index 27c5b0e..af68ebd 100644
> > --- a/gdb/source.c
> > +++ b/gdb/source.c
> > @@ -771,8 +771,28 @@ openp (const char *path, int opts, const char *string,
> >  	}
> >        else
> >  	{
> > -	  /* Normal file name in path -- just use it.  */
> > -	  strcpy (filename, dir);
> > +	  /* See whether we need to expand the tilde.  */
> > +	  if (strchr(dir, '~'))
> 
> 
> Seem to me this could be an "else if" above the "else" instead of
> nested within the "else".  The "Don't search $cdir" bit below
> should never apply, even in the rare beyond belief case of the user's
> home expanding to literal "$cdir".  Okay with that change.

You are right, here's what I committed.

-- 
[]'s
Thiago Jung Bauermann
Linaro Toolchain Working Group


2012-05-30  Thiago Jung Bauermann  <thiago.bauermann@linaro.org>
    
	* source.c (openp): Expand tilde in path entries.

diff --git a/gdb/source.c b/gdb/source.c
index 27c5b0e..7de86b4 100644
--- a/gdb/source.c
+++ b/gdb/source.c
@@ -769,6 +769,25 @@ openp (const char *path, int opts, const char *string,
 	    }
 	  strcpy (filename, current_directory);
 	}
+      else if (strchr(dir, '~'))
+	{
+	 /* See whether we need to expand the tilde.  */
+	  int newlen;
+	  char *tilde_expanded;
+
+	  tilde_expanded  = tilde_expand (dir);
+
+	  /* First, realloc the filename buffer if too short.  */
+	  len = strlen (tilde_expanded);
+	  newlen = len + strlen (string) + 2;
+	  if (newlen > alloclen)
+	    {
+	      alloclen = newlen;
+	      filename = alloca (alloclen);
+	    }
+	  strcpy (filename, tilde_expanded);
+	  xfree (tilde_expanded);
+	}
       else
 	{
 	  /* Normal file name in path -- just use it.  */