[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [git commit] do not duplicate CONFIG_PID_FILE_PATH and ".pid" strings
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2019-03-18 17:26:27
Message-ID: CAK1hOcMXhdXwKkpjMvfqMB5ar6iW6G+H6VAArnKqBJR+OpomxQ () mail ! gmail ! com
[Download RAW message or body]

On Mon, Mar 18, 2019 at 8:54 AM Bernhard Reutner-Fischer
<rep.dot.nop@gmail.com> wrote:
> On 17 March 2019 19:48:39 CET, Denys Vlasenko <vda.linux@googlemail.com> wrote:
> >commit:
> >https://git.busybox.net/busybox/commit/?id=5059653882dbd86e3bbf48389f9f81b0fac8cd0a
> >branch: https://git.busybox.net/busybox/commit/?id=refs/heads/master
> >
> >   text           data     bss     dec     hex filename
> > 981737            485    7296  989518   f194e busybox_old
> > 981704            485    7296  989485   f192d busybox_unstripped
> >
>
> >diff --git a/libbb/pidfile.c b/libbb/pidfile.c
> >index a48dfc38b..a6a282562 100644
> >--- a/libbb/pidfile.c
> >+++ b/libbb/pidfile.c
> >@@ -38,3 +38,21 @@ void FAST_FUNC write_pidfile(const char *path)
> >       }
> >       close(pid_fd);
> > }
> >+
> >+void FAST_FUNC write_pidfile_std_path_and_ext(const char *name)
> >+{
> >+      char buf[sizeof(CONFIG_PID_FILE_PATH) + 64];
> >+
> >+      snprintf(buf, sizeof(buf), CONFIG_PID_FILE_PATH"/%s.pid", name);
>
> concat_path_file()?

It can't append ".pid"
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]