How disgusting is this code?

Question

C++ / C#

Hello. How disgusting is this code? I have not yet tested it for performance, I just would like to know your opinion.

bool update_uwsgi_config() {
         const char *source = fmt::format("/tmp/{}.ini", unid).c_str();
         const char *on_restart = "service uwsgi restart";
         const char *dest = "/etc/uwsgi/apps-available/www.ini";
         const char *on_update = fmt::format("lxc file push {} {} {}", source,contname,dest).c_str();
         system(on_update);
         system(on_restart);
         remove(dest);
         return true;
     }

Answer 1

The main problem.
The fmt::format object will immediately disappear and source will hang.
Will have to write...

fmt::format fmSource("/tmp/{}.ini", unid);
const char *source = fmSource.c_str();

Etc.

Answer 2

- Not cross-platform (although it is hardly needed here if lxc is used)
- if contname or unid are accepted from the user, then this is a serious vulnerability
- why an extra dependency if formatting is in std::?
- it is better to put constants in the config, it is more convenient to edit them later
Yes, and this is usually written in python / bash, unless of course it is part of a project that is already in C ++