log.c:__lxc_log_set_file: fname cannot be null

fname cannot be passed in as NULL by any of its current callers.  If it
could, then build_dir() would crash as it doesn't check for it.  So make
sure we are warned if in the future we pass in NULL.

Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

diff --git a/src/lxc/log.c b/src/lxc/log.c
index 4796fe91ee94d6b0c41f281607d7f422c0fd4b01..948735d56563347dd2d9a77a0fcfdef3105a96be 100644 (file)
--- a/src/lxc/log.c
+++ b/src/lxc/log.c
@@ -20,6 +20,7 @@
  * License along with this library; if not, write to the Free Software
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
+#include <assert.h>
 #include <stdio.h>
 #include <errno.h>
 #include <limits.h>
@@ -266,7 +267,9 @@ static int __lxc_log_set_file(const char *fname, int create_dirs)
                free(log_fname);
        }
 
-       if (!fname || strlen(fname) == 0) {
+       assert(fname != NULL);
+
+       if (strlen(fname) == 0) {
                log_fname = NULL;
                return 0;
        }