From: Daniel Wagner <daniel.wagner(a)bmw-carit.de&gt; The comment doesn't provide any real additional information. It was documented why fd = -1 in the original commit. --- src/stats.c | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/stats.c b/src/stats.c index 98ce6d9ef741..eed6e8a1c09b 100644 --- a/src/stats.c +++ b/src/stats.c @@ -647,10 +647,6 @@ static int stats_file_history_update(struct stats_file *data_file) bzero(history_file, sizeof(struct stats_file)); bzero(temp_file, sizeof(struct stats_file)); - /* - * 0 is a valid file descriptor - fd needs to be initialized - * to -1 to handle errors correctly - */ history_file->fd = -1; temp_file->fd = -1; @@ -703,10 +699,6 @@ int __connman_stats_service_register(struct connman_service *service) if (!file) return -ENOMEM; - /* - * 0 is a valid file descriptor - fd needs to be initialized - * to -1 to handle errors correctly - */ file->fd = -1; g_hash_table_insert(stats_hash, service, file); -- 2.7.4