forked from pool/bcache-tools
100 lines
3.1 KiB
Diff
100 lines
3.1 KiB
Diff
|
From b0f5169a13976b05b183e0cde1fd23ef4f90101c Mon Sep 17 00:00:00 2001
|
||
|
From: Shaoxiong Li <dahefanteng@gmail.com>
|
||
|
Date: Wed, 19 Aug 2020 18:51:26 +0800
|
||
|
Subject: [PATCH 12/17] bcache-tools: Fix potential coredump issues
|
||
|
Git-commit: b0f5169a13976b05b183e0cde1fd23ef4f90101c
|
||
|
Patch-mainline: bcache-tools-1.1
|
||
|
References: jsc#SLE-9807
|
||
|
|
||
|
In some distributions, such as opensuse 15.2, when the free_dev
|
||
|
function is called, it may refer to the memory that has been
|
||
|
released, causing a coredump. Changing 'list_for_each_entry'
|
||
|
to 'list_for_each_entry_safe' can avoid this problem.
|
||
|
|
||
|
Signed-off-by: Shaoxiong Li <dahefanteng@gmail.com>
|
||
|
Signed-off-by: Coly Li <colyli@suse.de>
|
||
|
---
|
||
|
bcache.c | 18 +++++++++---------
|
||
|
1 file changed, 9 insertions(+), 9 deletions(-)
|
||
|
|
||
|
diff --git a/bcache.c b/bcache.c
|
||
|
index 50e3a88..3b963e4 100644
|
||
|
--- a/bcache.c
|
||
|
+++ b/bcache.c
|
||
|
@@ -175,9 +175,9 @@ int version_usagee(void)
|
||
|
|
||
|
void free_dev(struct list_head *head)
|
||
|
{
|
||
|
- struct dev *dev;
|
||
|
+ struct dev *dev, *n;
|
||
|
|
||
|
- list_for_each_entry(dev, head, dev_list) {
|
||
|
+ list_for_each_entry_safe(dev, n, head, dev_list) {
|
||
|
free(dev);
|
||
|
}
|
||
|
}
|
||
|
@@ -185,7 +185,7 @@ void free_dev(struct list_head *head)
|
||
|
int show_bdevs_detail(void)
|
||
|
{
|
||
|
struct list_head head;
|
||
|
- struct dev *devs;
|
||
|
+ struct dev *devs, *n;
|
||
|
|
||
|
INIT_LIST_HEAD(&head);
|
||
|
int ret;
|
||
|
@@ -197,7 +197,7 @@ int show_bdevs_detail(void)
|
||
|
}
|
||
|
printf("Name\t\tUuid\t\t\t\t\tCset_Uuid\t\t\t\tType\t\tState");
|
||
|
printf("\t\t\tBname\t\tAttachToDev\tAttachToCset\n");
|
||
|
- list_for_each_entry(devs, &head, dev_list) {
|
||
|
+ list_for_each_entry_safe(devs, n, &head, dev_list) {
|
||
|
printf("%s\t%s\t%s\t%lu", devs->name, devs->uuid,
|
||
|
devs->cset, devs->version);
|
||
|
switch (devs->version) {
|
||
|
@@ -242,7 +242,7 @@ int show_bdevs_detail(void)
|
||
|
int show_bdevs(void)
|
||
|
{
|
||
|
struct list_head head;
|
||
|
- struct dev *devs;
|
||
|
+ struct dev *devs, *n;
|
||
|
|
||
|
INIT_LIST_HEAD(&head);
|
||
|
int ret;
|
||
|
@@ -254,7 +254,7 @@ int show_bdevs(void)
|
||
|
}
|
||
|
|
||
|
printf("Name\t\tType\t\tState\t\t\tBname\t\tAttachToDev\n");
|
||
|
- list_for_each_entry(devs, &head, dev_list) {
|
||
|
+ list_for_each_entry_safe(devs, n, &head, dev_list) {
|
||
|
printf("%s\t%lu", devs->name, devs->version);
|
||
|
switch (devs->version) {
|
||
|
// These are handled the same by the kernel
|
||
|
@@ -428,7 +428,7 @@ int detail_single(char *devname)
|
||
|
int tree(void)
|
||
|
{
|
||
|
struct list_head head;
|
||
|
- struct dev *devs, *tmp;
|
||
|
+ struct dev *devs, *tmp, *n, *m;
|
||
|
|
||
|
INIT_LIST_HEAD(&head);
|
||
|
int ret;
|
||
|
@@ -445,13 +445,13 @@ int tree(void)
|
||
|
tb = scols_new_table();
|
||
|
scols_table_new_column(tb, ".", 0.1, SCOLS_FL_TREE);
|
||
|
scols_table_new_column(tb, "", 2, SCOLS_FL_TRUNC);
|
||
|
- list_for_each_entry(devs, &head, dev_list) {
|
||
|
+ list_for_each_entry_safe(devs, n, &head, dev_list) {
|
||
|
if ((devs->version == BCACHE_SB_VERSION_CDEV
|
||
|
|| devs->version == BCACHE_SB_VERSION_CDEV_WITH_UUID)
|
||
|
&& strcmp(devs->state, BCACHE_BASIC_STATE_ACTIVE) == 0) {
|
||
|
dad = scols_table_new_line(tb, NULL);
|
||
|
scols_line_set_data(dad, COL_CSET, devs->name);
|
||
|
- list_for_each_entry(tmp, &head, dev_list) {
|
||
|
+ list_for_each_entry_safe(tmp, m, &head, dev_list) {
|
||
|
if (strcmp(devs->cset, tmp->attachuuid) ==
|
||
|
0) {
|
||
|
son =
|
||
|
--
|
||
|
2.26.2
|
||
|
|