mirror of https://github.com/vitalif/e2fsprogs
misc: cppcheck cleanups
Fix a number of things that cppcheck complains about. Most of these are minor resource leaks and forgotten declarations. Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>crypto
parent
f1605243a4
commit
24997f1cd2
|
@ -666,7 +666,7 @@ static void dump_inline_data(FILE *out, const char *prefix, ext2_ino_t inode_num
|
|||
|
||||
retval = ext2fs_inline_data_size(current_fs, inode_num, &size);
|
||||
if (!retval)
|
||||
fprintf(out, "%sSize of inline data: %d", prefix, size);
|
||||
fprintf(out, "%sSize of inline data: %zu", prefix, size);
|
||||
}
|
||||
|
||||
void internal_dump_inode(FILE *out, const char *prefix,
|
||||
|
|
|
@ -177,6 +177,10 @@ extern time_t string_to_time(const char *arg);
|
|||
|
||||
/* xattrs.c */
|
||||
void dump_inode_attributes(FILE *out, ext2_ino_t ino);
|
||||
void do_get_xattr(int argc, char **argv);
|
||||
void do_set_xattr(int argc, char **argv);
|
||||
void do_rm_xattr(int argc, char **argv);
|
||||
void do_list_xattr(int argc, char **argv);
|
||||
|
||||
/* zap.c */
|
||||
extern void do_zap_block(int argc, char **argv);
|
||||
|
|
|
@ -446,7 +446,7 @@ static void check_is_really_dir(e2fsck_t ctx, struct problem_context *pctx,
|
|||
inlinedata_fs = (ctx->fs->super->s_feature_incompat &
|
||||
EXT4_FEATURE_INCOMPAT_INLINE_DATA);
|
||||
if (inlinedata_fs && (inode->i_flags & EXT4_INLINE_DATA_FL)) {
|
||||
unsigned int size;
|
||||
size_t size;
|
||||
|
||||
if (ext2fs_inline_data_size(ctx->fs, pctx->ino, &size))
|
||||
return;
|
||||
|
|
|
@ -770,7 +770,7 @@ static int check_dir_block(ext2_filsys fs,
|
|||
int dx_csum_size = 0, de_csum_size = 0;
|
||||
int failed_csum = 0;
|
||||
int is_leaf = 1;
|
||||
int inline_data_size = 0;
|
||||
size_t inline_data_size = 0;
|
||||
int filetype = 0;
|
||||
|
||||
cd = (struct check_dir_struct *) priv_data;
|
||||
|
|
|
@ -203,6 +203,8 @@ int main(int argc, char **argv)
|
|||
break;
|
||||
}
|
||||
}
|
||||
if (r)
|
||||
region_free(r);
|
||||
}
|
||||
|
||||
#endif /* TEST_PROGRAM */
|
||||
|
|
|
@ -18,6 +18,7 @@
|
|||
|
||||
#include "ext2_fs.h"
|
||||
#include "ext2fs.h"
|
||||
#include "ext2fsP.h"
|
||||
|
||||
struct expand_dir_struct {
|
||||
int done;
|
||||
|
|
|
@ -1386,6 +1386,17 @@ errcode_t ext2fs_icount_validate(ext2_icount_t icount, FILE *);
|
|||
extern errcode_t ext2fs_get_memalign(unsigned long size,
|
||||
unsigned long align, void *ptr);
|
||||
|
||||
/* inline_data.c */
|
||||
extern errcode_t ext2fs_inline_data_init(ext2_filsys fs, ext2_ino_t ino);
|
||||
extern errcode_t ext2fs_inline_data_size(ext2_filsys fs, ext2_ino_t ino,
|
||||
size_t *size);
|
||||
extern errcode_t ext2fs_inline_data_get(ext2_filsys fs, ext2_ino_t ino,
|
||||
struct ext2_inode *inode,
|
||||
void *buf, size_t *size);
|
||||
extern errcode_t ext2fs_inline_data_set(ext2_filsys fs, ext2_ino_t ino,
|
||||
struct ext2_inode *inode,
|
||||
void *buf, size_t size);
|
||||
|
||||
/* inode.c */
|
||||
extern errcode_t ext2fs_create_inode_cache(ext2_filsys fs,
|
||||
unsigned int cache_size);
|
||||
|
|
|
@ -88,20 +88,11 @@ extern int ext2fs_process_dir_block(ext2_filsys fs,
|
|||
int ref_offset,
|
||||
void *priv_data);
|
||||
|
||||
extern errcode_t ext2fs_inline_data_init(ext2_filsys fs, ext2_ino_t ino);
|
||||
extern errcode_t ext2fs_inline_data_size(ext2_filsys fs, ext2_ino_t ino,
|
||||
size_t *size);
|
||||
extern errcode_t ext2fs_inline_data_ea_remove(ext2_filsys fs, ext2_ino_t ino);
|
||||
extern errcode_t ext2fs_inline_data_expand(ext2_filsys fs, ext2_ino_t ino);
|
||||
extern int ext2fs_inline_data_dir_iterate(ext2_filsys fs,
|
||||
ext2_ino_t ino,
|
||||
void *priv_data);
|
||||
extern errcode_t ext2fs_inline_data_get(ext2_filsys fs, ext2_ino_t ino,
|
||||
struct ext2_inode *inode,
|
||||
void *buf, size_t *size);
|
||||
extern errcode_t ext2fs_inline_data_set(ext2_filsys fs, ext2_ino_t ino,
|
||||
struct ext2_inode *inode,
|
||||
void *buf, size_t size);
|
||||
|
||||
/* Generic numeric progress meter */
|
||||
|
||||
|
|
|
@ -26,6 +26,7 @@
|
|||
|
||||
#include "ext2_fs.h"
|
||||
#include "ext2fs.h"
|
||||
#include "ext2fsP.h"
|
||||
|
||||
#ifndef EXT2_FT_DIR
|
||||
#define EXT2_FT_DIR 2
|
||||
|
|
|
@ -19,6 +19,7 @@
|
|||
|
||||
#include "ext2_fs.h"
|
||||
#include "ext2fs.h"
|
||||
#include "ext2fsP.h"
|
||||
|
||||
#undef PUNCH_DEBUG
|
||||
|
||||
|
|
|
@ -426,6 +426,8 @@ int main(int argc, char **argv)
|
|||
}
|
||||
if (old)
|
||||
fclose(old);
|
||||
if (newfn)
|
||||
free(newfn);
|
||||
return (0);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue