diff options
author | Karel Zak | 2012-12-04 14:22:39 +0100 |
---|---|---|
committer | Karel Zak | 2012-12-04 14:22:39 +0100 |
commit | ba2bdf41c436640286df40529cddede46b3ba5d8 (patch) | |
tree | 2fd343c6a2450b4566ca996ff2f0c9703bc2e680 /libmount/src/tab_parse.c | |
parent | libblkid: re-read PART{UUID,LABEL} for successfully verified cached devices (diff) | |
download | kernel-qcow2-util-linux-ba2bdf41c436640286df40529cddede46b3ba5d8.tar.gz kernel-qcow2-util-linux-ba2bdf41c436640286df40529cddede46b3ba5d8.tar.xz kernel-qcow2-util-linux-ba2bdf41c436640286df40529cddede46b3ba5d8.zip |
libmount: clean nonnull attribute usage
- use __attribute__((nonnull) for functions where we not able to
return an return code ("is", "has" and some "get" functions).
- use __attribute__((nonnull) for small functions where we
always modify any of the function argument (some mnt_optstr_* functions)
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'libmount/src/tab_parse.c')
-rw-r--r-- | libmount/src/tab_parse.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libmount/src/tab_parse.c b/libmount/src/tab_parse.c index 8a9d7349e..56da8ec47 100644 --- a/libmount/src/tab_parse.c +++ b/libmount/src/tab_parse.c @@ -791,7 +791,8 @@ struct libmnt_table *mnt_new_table_from_dir(const char *dirname) int mnt_table_set_parser_errcb(struct libmnt_table *tb, int (*cb)(struct libmnt_table *tb, const char *filename, int line)) { - assert(tb); + if (!tb) + return -EINVAL; tb->errcb = cb; return 0; } @@ -804,7 +805,8 @@ int mnt_table_set_parser_fltrcb(struct libmnt_table *tb, int (*cb)(struct libmnt_fs *, void *), void *data) { - assert(tb); + if (!tb) + return -EINVAL; DBG(TAB, mnt_debug_h(tb, "%s table parser filter", cb ? "set" : "unset")); tb->fltrcb = cb; |