As well as what BrowserUk said, you should be using closedir to close a directory handle opened by opendir instead of close which works on filehandles instead.