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~