[rcthompson]

Shouldn't you throw in a `sync` before halting?

[Nick_C]

If you do that, I would change && to ; on the basis you want the halt no matter whether the umount succeeds or not.