Re: [Tracker] Bad use of /tmp




It is unique when you keep the same trackerd all over session but it
would be very rare to see trackerd get one of its old pid after it
exited or crashed.

Well, using the PIDs, it would be safe to remove any dir existing on startup. But hey. It's basically the tomato/tomatoe discussion. ;)

> Still, good idea, and the patch looks clean while just scanning it.
> Though I might recommend looking over the first line of
> sanity_check_option_values() in src/trackerd/trackerd.c. The point of
> the patch seems overall to get rid of g_get_tmp_dir(). :)

I use g_get_tmp_dir() and try to make a temporary directory into it.

But I send a new patch a little bit cleaner.
*ahh* I see that now. Looks just a tidy bit more coherent now, though. :)


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]