Changeset 5e801dc in mainline for uspace/srv/fs/mfs/mfs_ops.c

Timestamp:

2019-02-25T14:42:38Z (6 years ago)

Author:

GitHub <noreply@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

a4e78743

Parents:

ee8d4d6

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2019-02-25 14:42:38)

git-committer:

GitHub <noreply@…> (2019-02-25 14:42:38)

Message:

Indicate and enforce constness of hash table key in certain functions (#158)

The assumption here is that modifying key in the hash/equal functions in something completely unexpected, and not something you would ever want to do intentionally, so it makes sense to disallow it entirely to get that extra level of checking.

File:

• uspace/srv/fs/mfs/mfs_ops.c (modified) (2 diffs)

uspace/srv/fs/mfs/mfs_ops.c

@@ -100 +100 @@
 
 static size_t
-open_nodes_key_hash(void *key)
-{
-        node_key_t *node_key = (node_key_t *)key;
+open_nodes_key_hash(const void *key)
+{
+        const node_key_t *node_key = key;
         return hash_combine(node_key->service_id, node_key->index);
 }
@@ -114 +114 @@
 
 static bool
-open_nodes_key_equal(void *key, const ht_link_t *item)
-{
-        node_key_t *node_key = (node_key_t *)key;
+open_nodes_key_equal(const void *key, const ht_link_t *item)
+{
+        const node_key_t *node_key = key;
         struct mfs_node *mnode = hash_table_get_inst(item, struct mfs_node, link);