I'm puzzled.  Given that line 602 appends a /. to GetLogDir() and 603 makes its parent, surely this is redundant; or belongs before 602.