mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2024-12-25 18:59:33 +03:00
72c99dc959
GCC 12.1.1_p20220625's static analyzer caught these. Of the two in the btree test, one had previously been caught by Coverity and Smatch, but GCC flagged it as a false positive. Upon examining how other test cases handle this, the solution was changed from `ASSERT3P(node, !=, NULL);` to using `perror()` to be consistent with the fixes to the other fixes done to the ZTS code. That approach was also used in ZED since I did not see a better way of handling this there. Also, upon inspection, additional unchecked pointers from malloc()/calloc()/strdup() were found in ZED, so those were handled too. In other parts of the code, the existing methods to avoid issues from memory allocators returning NULL were used, such as using `umem_alloc(size, UMEM_NOFAIL)` or returning `ENOMEM`. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Reviewed-by: Tony Hutter <hutter2@llnl.gov> Signed-off-by: Richard Yao <richard.yao@alumni.stonybrook.edu> Closes #13979 |
||
|---|---|---|
| .. | ||
| agents | ||
| zed.d | ||
| .gitignore | ||
| Makefile.am | ||
| zed_conf.c | ||
| zed_conf.h | ||
| zed_disk_event.c | ||
| zed_disk_event.h | ||
| zed_event.c | ||
| zed_event.h | ||
| zed_exec.c | ||
| zed_exec.h | ||
| zed_file.c | ||
| zed_file.h | ||
| zed_log.c | ||
| zed_log.h | ||
| zed_strings.c | ||
| zed_strings.h | ||
| zed.c | ||
| zed.h | ||