Re: [PATCH 04/51] semihosting/arm-compat-semi: Avoid using hardcoded /tmp

[Richard Henderson]

On 9/1/22 08:11, Bin Meng wrote:
> > > -        len = asprintf(&s, "/tmp/qemu-%x%02x", getpid(), (int)arg1 & 0xff);
> > > +        len = asprintf(&s, "%s/qemu-%x%02x", g_get_tmp_dir(),
> > > +                       getpid(), (int)arg1 & 0xff);
> >
> >
> > This is most likely wrong. I am not familiar with semihosting, but I believe we 
> > are implementing tmpnam(), it should return a POSIX filename.
>
> Replacing /tmp with g_get_tmp_dir() is not wrong, correct?

The replacement is correct.


r~