Clean up: Since e_fslocdata is allocated with strdup(3), and not xstrdup(), it should be freed with free(3), and not xfree(). It looks like we could get rid of the "if (thing I'm freeing isn't NULL)" checks too, but that's for another day. Signed-off-by: Chuck Lever <chuck.lever@xxxxxxxxxx> --- support/export/export.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/support/export/export.c b/support/export/export.c index ba643b2..42e78f6 100644 --- a/support/export/export.c +++ b/support/export/export.c @@ -277,7 +277,7 @@ export_freeall(void) if (exp->m_export.e_mountpoint) free(exp->m_export.e_mountpoint); if (exp->m_export.e_fslocdata) - xfree(exp->m_export.e_fslocdata); + free(exp->m_export.e_fslocdata); xfree(exp->m_export.e_hostname); xfree(exp); } -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
