dgaudet 98/06/12 10:42:43
Modified: src/modules/standard mod_autoindex.c Log: - It's not possible for name to be NULL. - It's not necessary to test name[0] == 0 because the next if will fail in that event anyhow. Ditto for name[1] == 0. Revision Changes Path 1.80 +1 -13 apache-1.3/src/modules/standard/mod_autoindex.c Index: mod_autoindex.c =================================================================== RCS file: /export/home/cvs/apache-1.3/src/modules/standard/mod_autoindex.c,v retrieving revision 1.79 retrieving revision 1.80 diff -u -r1.79 -r1.80 --- mod_autoindex.c 1998/06/12 15:53:23 1.79 +++ mod_autoindex.c 1998/06/12 17:42:42 1.80 @@ -140,20 +140,8 @@ * matches ".." or "../"). Hopefully this one call is significantly less * expensive than multiple strcmp() calls. */ -static int is_parent(const char *name) +static ap_inline int is_parent(const char *name) { - /* - * If it's no name at all, it isn't our parent. - */ - if (name == NULL) { - return 0; - } - /* - * Nor is it if the name isn't long enough. - */ - if ((name[0] == '\0') || (name[1] == '\0')) { - return 0; - } /* * Now, IFF the first two bytes are dots, and the third byte is either * EOS (\0) or a slash followed by EOS, we have a match.