[screen-devel] [bug #38335] /tmp/screens creation race

[Richard Kojedzinszky]

URL:
  <http://savannah.gnu.org/bugs/?38335>

                 Summary: /tmp/screens creation race
                 Project: GNU Screen
            Submitted by: krichy
            Submitted on: Fri 15 Feb 2013 05:33:30 AM GMT
                Category: Program Logic
                Severity: 3 - Normal
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
                 Release: 4.0.3
           Fixed Release: None
         Planned Release: None
           Work Required: None

    _______________________________________________________

Details:

Our unix user accounts for different services have a @reboot entry in their
crontabs, but as cron starts this entries paralell upon boot, I eventually got
some screen instances not starting, and the corresponding error message is
sent by cron, stating that /tmp/screens already exists.

It seems that first a check is made for existence of that dir (stat()), and if
missing, then it is created. In my opinion, an mkdir() could always be called
instead stat() and mkdir(), as it implicitly checks the existence, actually
does no harm at all, and the result is the same. Or, to finally check the
rights on the dir, then they may be reversed, so first mkdir(), then stat()
it.





    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?38335>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/