Changeset 5e801dc in mainline for uspace/lib/c/generic/async/ports.c

Timestamp:

2019-02-25T14:42:38Z (5 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/lib/c/generic/async/ports.c (modified) (4 diffs)

uspace/lib/c/generic/async/ports.c

@@ -103 +103 @@
 static hash_table_t interface_hash_table;
 
-static size_t interface_key_hash(void *key)
-{
-        iface_t iface = *(iface_t *) key;
-        return iface;
+static size_t interface_key_hash(const void *key)
+{
+        const iface_t *iface = key;
+        return *iface;
 }
 
@@ -115 +115 @@
 }
 
-static bool interface_key_equal(void *key, const ht_link_t *item)
-{
-        iface_t iface = *(iface_t *) key;
+static bool interface_key_equal(const void *key, const ht_link_t *item)
+{
+        const iface_t *iface = key;
         interface_t *interface = hash_table_get_inst(item, interface_t, link);
-        return iface == interface->iface;
+        return *iface == interface->iface;
 }
 
@@ -131 +131 @@
 };
 
-static size_t port_key_hash(void *key)
-{
-        port_id_t port_id = *(port_id_t *) key;
-        return port_id;
+static size_t port_key_hash(const void *key)
+{
+        const port_id_t *port_id = key;
+        return *port_id;
 }
 
@@ -143 +143 @@
 }
 
-static bool port_key_equal(void *key, const ht_link_t *item)
-{
-        port_id_t port_id = *(port_id_t *) key;
+static bool port_key_equal(const void *key, const ht_link_t *item)
+{
+        const port_id_t *port_id = key;
         port_t *port = hash_table_get_inst(item, port_t, link);
-        return port_id == port->id;
+        return *port_id == port->id;
 }