Properly patch up dirname()/basename() calls to not clobber ent->log.

[ed]

Summary

It turns out that we had a couple of more calls to dirname()/basename()
in newsyslog(8) that assume the input isn't clobbered. This is bad,
because it apparently breaks log rotation now that the new dirname()
implementation has been merged.

Fix this by first copying the input and then calling
dirname()/basename(). While there, improve the naming of variables in
this function a bit.

[bdrewery]

usr.sbin/newsyslog/newsyslog.c

1535

I don't see a reason for this old strdup