Changeset 8a64320e in mainline for uspace/lib

Timestamp:

2015-04-23T23:40:14Z (11 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

dcba819

Parents:

09044cb

Message:

pre-merge coding style cleanup and code review

Location:

uspace/lib

Files:

• c/include/ieee80211/ieee80211.h (modified) (2 diffs)
• crypto/aes.c (modified) (20 diffs)
• crypto/crypto.c (modified) (12 diffs)
• crypto/crypto.h (modified) (1 diff)
• crypto/rc4.c (modified) (5 diffs)
• drv/generic/remote_ieee80211.c (modified) (15 diffs)
• drv/include/ieee80211_iface.h (modified) (2 diffs)
• drv/include/ops/ieee80211.h (modified) (1 diff)
• ieee80211/Makefile (modified) (1 diff)
• ieee80211/include/ieee80211.h (modified) (4 diffs)
• ieee80211/include/ieee80211_iface_impl.h (modified) (2 diffs)
• ieee80211/include/ieee80211_impl.h (modified) (2 diffs)
• ieee80211/include/ieee80211_private.h (modified) (12 diffs)
• ieee80211/src/ieee80211.c (modified) (67 diffs)
• ieee80211/src/ieee80211_iface_impl.c (modified) (9 diffs)
• ieee80211/src/ieee80211_impl.c (modified) (19 diffs)

uspace/lib/c/include/ieee80211/ieee80211.h

@@ -44 +44 @@
 
 /** Max length of scan results array. */
-#define IEEE80211_MAX_RESULTS_LENGTH 32
+#define IEEE80211_MAX_RESULTS_LENGTH  32
 
 /** Max SSID length including null character. */
-#define IEEE80211_MAX_SSID_LENGTH 35
+#define IEEE80211_MAX_SSID_LENGTH  35
 
 /** WiFi security authentication method indicator. */
@@ -97 +97 @@
 /** @}
  */
-

uspace/lib/crypto/aes.c

@@ -28 +28 @@
 
 /** @file aes.c
- * 
+ *
  * Implementation of AES-128 symmetric cipher cryptographic algorithm.
- * 
+ *
  * Based on FIPS 197.
  */
@@ -37 +37 @@
 #include <errno.h>
 #include <mem.h>
-
 #include "crypto.h"
 
 /* Number of elements in rows/columns in AES arrays. */
-#define ELEMS 4
+#define ELEMS  4
 
 /* Number of elements (words) in cipher. */
-#define CIPHER_ELEMS 4
+#define CIPHER_ELEMS  4
 
 /* Length of AES block. */
-#define BLOCK_LEN 16
+#define BLOCK_LEN  16
 
 /* Number of iterations in AES algorithm. */
-#define ROUNDS 10
-
-/* 
- * Irreducible polynomial used in AES algorithm. 
- * 
+#define ROUNDS  10
+
+/** Irreducible polynomial used in AES algorithm.
+ *
  * NOTE: x^8 + x^4 + x^3 + x + 1.
- */
-#define AES_IP 0x1B
-
-/* Precomputed values for AES sub_byte transformation. */
+ *
+ */
+#define AES_IP  0x1b
+
+/** Precomputed values for AES sub_byte transformation. */
 static const uint8_t sbox[BLOCK_LEN][BLOCK_LEN] = {
         {
-                0x63, 0x7c, 0x77, 0x7b, 0xf2, 0x6b, 0x6f, 0xc5, 
+                0x63, 0x7c, 0x77, 0x7b, 0xf2, 0x6b, 0x6f, 0xc5,
                 0x30, 0x01, 0x67, 0x2b, 0xfe, 0xd7, 0xab, 0x76
         },
         {
-                0xca, 0x82, 0xc9, 0x7d, 0xfa, 0x59, 0x47, 0xf0, 
+                0xca, 0x82, 0xc9, 0x7d, 0xfa, 0x59, 0x47, 0xf0,
                 0xad, 0xd4, 0xa2, 0xaf, 0x9c, 0xa4, 0x72, 0xc0
         },
@@ -74 +73 @@
         },
         {
-                0x04, 0xc7, 0x23, 0xc3, 0x18, 0x96, 0x05, 0x9a, 
+                0x04, 0xc7, 0x23, 0xc3, 0x18, 0x96, 0x05, 0x9a,
                 0x07, 0x12, 0x80, 0xe2, 0xeb, 0x27, 0xb2, 0x75
         },
         {
-                0x09, 0x83, 0x2c, 0x1a, 0x1b, 0x6e, 0x5a, 0xa0, 
+                0x09, 0x83, 0x2c, 0x1a, 0x1b, 0x6e, 0x5a, 0xa0,
                 0x52, 0x3b, 0xd6, 0xb3, 0x29, 0xe3, 0x2f, 0x84
         },
         {
-                0x53, 0xd1, 0x00, 0xed, 0x20, 0xfc, 0xb1, 0x5b, 
+                0x53, 0xd1, 0x00, 0xed, 0x20, 0xfc, 0xb1, 0x5b,
                 0x6a, 0xcb, 0xbe, 0x39, 0x4a, 0x4c, 0x58, 0xcf
         },
         {
-                0xd0, 0xef, 0xaa, 0xfb, 0x43, 0x4d, 0x33, 0x85, 
+                0xd0, 0xef, 0xaa, 0xfb, 0x43, 0x4d, 0x33, 0x85,
                 0x45, 0xf9, 0x02, 0x7f, 0x50, 0x3c, 0x9f, 0xa8
         },
         {
-                0x51, 0xa3, 0x40, 0x8f, 0x92, 0x9d, 0x38, 0xf5, 
+                0x51, 0xa3, 0x40, 0x8f, 0x92, 0x9d, 0x38, 0xf5,
                 0xbc, 0xb6, 0xda, 0x21, 0x10, 0xff, 0xf3, 0xd2
         },
         {
-                0xcd, 0x0c, 0x13, 0xec, 0x5f, 0x97, 0x44, 0x17, 
+                0xcd, 0x0c, 0x13, 0xec, 0x5f, 0x97, 0x44, 0x17,
                 0xc4, 0xa7, 0x7e, 0x3d, 0x64, 0x5d, 0x19, 0x73
         },
         {
-                0x60, 0x81, 0x4f, 0xdc, 0x22, 0x2a, 0x90, 0x88, 
+                0x60, 0x81, 0x4f, 0xdc, 0x22, 0x2a, 0x90, 0x88,
                 0x46, 0xee, 0xb8, 0x14, 0xde, 0x5e, 0x0b, 0xdb
         },
         {
-                0xe0, 0x32, 0x3a, 0x0a, 0x49, 0x06, 0x24, 0x5c, 
+                0xe0, 0x32, 0x3a, 0x0a, 0x49, 0x06, 0x24, 0x5c,
                 0xc2, 0xd3, 0xac, 0x62, 0x91, 0x95, 0xe4, 0x79
         },
         {
-                0xe7, 0xc8, 0x37, 0x6d, 0x8d, 0xd5, 0x4e, 0xa9, 
+                0xe7, 0xc8, 0x37, 0x6d, 0x8d, 0xd5, 0x4e, 0xa9,
                 0x6c, 0x56, 0xf4, 0xea, 0x65, 0x7a, 0xae, 0x08
         },
         {
-                0xba, 0x78, 0x25, 0x2e, 0x1c, 0xa6, 0xb4, 0xc6, 
+                0xba, 0x78, 0x25, 0x2e, 0x1c, 0xa6, 0xb4, 0xc6,
                 0xe8, 0xdd, 0x74, 0x1f, 0x4b, 0xbd, 0x8b, 0x8a
         },
         {
-                0x70, 0x3e, 0xb5, 0x66, 0x48, 0x03, 0xf6, 0x0e, 
+                0x70, 0x3e, 0xb5, 0x66, 0x48, 0x03, 0xf6, 0x0e,
                 0x61, 0x35, 0x57, 0xb9, 0x86, 0xc1, 0x1d, 0x9e
         },
         {
-                0xe1, 0xf8, 0x98, 0x11, 0x69, 0xd9, 0x8e, 0x94, 
+                0xe1, 0xf8, 0x98, 0x11, 0x69, 0xd9, 0x8e, 0x94,
                 0x9b, 0x1e, 0x87, 0xe9, 0xce, 0x55, 0x28, 0xdf
         },
         {
-                0x8c, 0xa1, 0x89, 0x0d, 0xbf, 0xe6, 0x42, 0x68, 
+                0x8c, 0xa1, 0x89, 0x0d, 0xbf, 0xe6, 0x42, 0x68,
                 0x41, 0x99, 0x2d, 0x0f, 0xb0, 0x54, 0xbb, 0x16
         }
 };
 
-/* Precomputed values for AES inv_sub_byte transformation. */
+/** Precomputed values for AES inv_sub_byte transformation. */
 static uint8_t inv_sbox[BLOCK_LEN][BLOCK_LEN] = {
         {
-                0x52, 0x09, 0x6a, 0xd5, 0x30, 0x36, 0xa5, 0x38, 
+                0x52, 0x09, 0x6a, 0xd5, 0x30, 0x36, 0xa5, 0x38,
                 0xbf, 0x40, 0xa3, 0x9e, 0x81, 0xf3, 0xd7, 0xfb
         },
         {
-                0x7c, 0xe3, 0x39, 0x82, 0x9b, 0x2f, 0xff, 0x87, 
+                0x7c, 0xe3, 0x39, 0x82, 0x9b, 0x2f, 0xff, 0x87,
                 0x34, 0x8e, 0x43, 0x44, 0xc4, 0xde, 0xe9, 0xcb
         },
         {
-                0x54, 0x7b, 0x94, 0x32, 0xa6, 0xc2, 0x23, 0x3d, 
+                0x54, 0x7b, 0x94, 0x32, 0xa6, 0xc2, 0x23, 0x3d,
                 0xee, 0x4c, 0x95, 0x0b, 0x42, 0xfa, 0xc3, 0x4e
         },
         {
-                0x08, 0x2e, 0xa1, 0x66, 0x28, 0xd9, 0x24, 0xb2, 
+                0x08, 0x2e, 0xa1, 0x66, 0x28, 0xd9, 0x24, 0xb2,
                 0x76, 0x5b, 0xa2, 0x49, 0x6d, 0x8b, 0xd1, 0x25
         },
         {
-                0x72, 0xf8, 0xf6, 0x64, 0x86, 0x68, 0x98, 0x16, 
+                0x72, 0xf8, 0xf6, 0x64, 0x86, 0x68, 0x98, 0x16,
                 0xd4, 0xa4, 0x5c, 0xcc, 0x5d, 0x65, 0xb6, 0x92
         },
         {
-                0x6c, 0x70, 0x48, 0x50, 0xfd, 0xed, 0xb9, 0xda, 
+                0x6c, 0x70, 0x48, 0x50, 0xfd, 0xed, 0xb9, 0xda,
                 0x5e, 0x15, 0x46, 0x57, 0xa7, 0x8d, 0x9d, 0x84
         },
         {
-                0x90, 0xd8, 0xab, 0x00, 0x8c, 0xbc, 0xd3, 0x0a, 
+                0x90, 0xd8, 0xab, 0x00, 0x8c, 0xbc, 0xd3, 0x0a,
                 0xf7, 0xe4, 0x58, 0x05, 0xb8, 0xb3, 0x45, 0x06
         },
         {
-                0xd0, 0x2c, 0x1e, 0x8f, 0xca, 0x3f, 0x0f, 0x02, 
+                0xd0, 0x2c, 0x1e, 0x8f, 0xca, 0x3f, 0x0f, 0x02,
                 0xc1, 0xaf, 0xbd, 0x03, 0x01, 0x13, 0x8a, 0x6b
         },
         {
-                0x3a, 0x91, 0x11, 0x41, 0x4f, 0x67, 0xdc, 0xea, 
+                0x3a, 0x91, 0x11, 0x41, 0x4f, 0x67, 0xdc, 0xea,
                 0x97, 0xf2, 0xcf, 0xce, 0xf0, 0xb4, 0xe6, 0x73
         },
         {
-                0x96, 0xac, 0x74, 0x22, 0xe7, 0xad, 0x35, 0x85, 
+                0x96, 0xac, 0x74, 0x22, 0xe7, 0xad, 0x35, 0x85,
                 0xe2, 0xf9, 0x37, 0xe8, 0x1c, 0x75, 0xdf, 0x6e
         },
         {
-                0x47, 0xf1, 0x1a, 0x71, 0x1d, 0x29, 0xc5, 0x89, 
+                0x47, 0xf1, 0x1a, 0x71, 0x1d, 0x29, 0xc5, 0x89,
                 0x6f, 0xb7, 0x62, 0x0e, 0xaa, 0x18, 0xbe, 0x1b
         },
         {
-                0xfc, 0x56, 0x3e, 0x4b, 0xc6, 0xd2, 0x79, 0x20, 
+                0xfc, 0x56, 0x3e, 0x4b, 0xc6, 0xd2, 0x79, 0x20,
                 0x9a, 0xdb, 0xc0, 0xfe, 0x78, 0xcd, 0x5a, 0xf4
         },
         {
-                0x1f, 0xdd, 0xa8, 0x33, 0x88, 0x07, 0xc7, 0x31, 
+                0x1f, 0xdd, 0xa8, 0x33, 0x88, 0x07, 0xc7, 0x31,
                 0xb1, 0x12, 0x10, 0x59, 0x27, 0x80, 0xec, 0x5f
         },
         {
-                0x60, 0x51, 0x7f, 0xa9, 0x19, 0xb5, 0x4a, 0x0d, 
+                0x60, 0x51, 0x7f, 0xa9, 0x19, 0xb5, 0x4a, 0x0d,
                 0x2d, 0xe5, 0x7a, 0x9f, 0x93, 0xc9, 0x9c, 0xef
         },
         {
-                0xa0, 0xe0, 0x3b, 0x4d, 0xae, 0x2a, 0xf5, 0xb0, 
+                0xa0, 0xe0, 0x3b, 0x4d, 0xae, 0x2a, 0xf5, 0xb0,
                 0xc8, 0xeb, 0xbb, 0x3c, 0x83, 0x53, 0x99, 0x61
         },
         {
-                0x17, 0x2b, 0x04, 0x7e, 0xba, 0x77, 0xd6, 0x26, 
+                0x17, 0x2b, 0x04, 0x7e, 0xba, 0x77, 0xd6, 0x26,
                 0xe1, 0x69, 0x14, 0x63, 0x55, 0x21, 0x0c, 0x7d
         }
 };
 
-/* Precomputed values of powers of 2 in GF(2^8) left shifted by 24b. */
+/** Precomputed values of powers of 2 in GF(2^8) left shifted by 24b. */
 static const uint32_t r_con_array[] = {
-        0x01000000, 0x02000000, 0x04000000, 0x08000000, 
+        0x01000000, 0x02000000, 0x04000000, 0x08000000,
         0x10000000, 0x20000000, 0x40000000, 0x80000000,
-        0x1B000000, 0x36000000
+        0x1b000000, 0x36000000
 };
 
-/**
- * Perform substitution transformation on given byte.
- * 
+/** Perform substitution transformation on given byte.
+ *
  * @param byte Input byte.
- * @param inv Flag indicating whether to use inverse table.
- * 
+ * @param inv  Flag indicating whether to use inverse table.
+ *
  * @return Substituted value.
+ *
  */
 static uint8_t sub_byte(uint8_t byte, bool inv)
@@ -215 +214 @@
         uint8_t j = byte & 0xF;
         
-        if(!inv) {
+        if (!inv)
                 return sbox[i][j];
-        } else {
-                return inv_sbox[i][j];
-        }
-}
-
-/**
- * Perform substitution transformation on state table.
- * 
+        
+        return inv_sbox[i][j];
+}
+
+/** Perform substitution transformation on state table.
+ *
  * @param state State table to be modified.
- * @param inv Flag indicating whether to use inverse table.
+ * @param inv   Flag indicating whether to use inverse table.
+ *
  */
 static void sub_bytes(uint8_t state[ELEMS][ELEMS], bool inv)
@@ -232 +230 @@
         uint8_t val;
         
-        for(size_t i = 0; i < ELEMS; i++) {
-                for(size_t j = 0; j < ELEMS; j++) {
+        for (size_t i = 0; i < ELEMS; i++) {
+                for (size_t j = 0; j < ELEMS; j++) {
                         val = state[i][j];
                         state[i][j] = sub_byte(val, inv);
@@ -240 +238 @@
 }
 
-/**
- * Perform shift rows transformation on state table.
- * 
+/** Perform shift rows transformation on state table.
+ *
  * @param state State table to be modified.
+ *
  */
 static void shift_rows(uint8_t state[ELEMS][ELEMS])
@@ -249 +247 @@
         uint8_t temp[ELEMS];
         
-        for(size_t i = 1; i < ELEMS; i++) {
+        for (size_t i = 1; i < ELEMS; i++) {
                 memcpy(temp, state[i], i);
                 memcpy(state[i], state[i] + i, ELEMS - i);
@@ -256 +254 @@
 }
 
-/**
- * Perform inverted shift rows transformation on state table.
- * 
+/** Perform inverted shift rows transformation on state table.
+ *
  * @param state State table to be modified.
+ *
  */
 static void inv_shift_rows(uint8_t state[ELEMS][ELEMS])
@@ -265 +263 @@
         uint8_t temp[ELEMS];
         
-        for(size_t i = 1; i < ELEMS; i++) {
+        for (size_t i = 1; i < ELEMS; i++) {
                 memcpy(temp, state[i], ELEMS - i);
                 memcpy(state[i], state[i] + ELEMS - i, i);
@@ -272 +270 @@
 }
 
-/**
- * Multiplication in GF(2^8).
- * 
+/** Multiplication in GF(2^8).
+ *
  * @param x First factor.
  * @param y Second factor.
- * 
+ *
  * @return Multiplication of given factors in GF(2^8).
- */
-static uint8_t galois_mult(uint8_t x, uint8_t y) {
-        uint8_t result = 0;
-        uint8_t F_bitH;
-        
-        for(size_t i = 0; i < 8; i++) {
-                if (y & 1) 
+ *
+ */
+static uint8_t galois_mult(uint8_t x, uint8_t y)
+{
+        uint8_t result = 0;
+        uint8_t f_bith;
+        
+        for (size_t i = 0; i < 8; i++) {
+                if (y & 1)
                         result ^= x;
-                F_bitH = (x & 0x80);
-                x <<= 1;
-                if (F_bitH) 
-                        x ^= AES_IP;
-                y >>= 1;
-        }
-        
-        return result;
-}
-
-/**
- * Perform mix columns transformation on state table.
- * 
+                
+                f_bith = (x & 0x80);
+                x <<= 1;
+                
+                if (f_bith)
+                        x ^= AES_IP;
+                
+                y >>= 1;
+        }
+        
+        return result;
+}
+
+/** Perform mix columns transformation on state table.
+ *
  * @param state State table to be modified.
+ *
  */
 static void mix_columns(uint8_t state[ELEMS][ELEMS])
@@ -307 +309 @@
         memcpy(orig_state, state, BLOCK_LEN);
         
-        for(size_t j = 0; j < ELEMS; j++) {
-                state[0][j] = 
-                        galois_mult(0x2, orig_state[0][j]) ^
-                        galois_mult(0x3, orig_state[1][j]) ^
-                        orig_state[2][j] ^
-                        orig_state[3][j];
-                state[1][j] = 
-                        orig_state[0][j] ^
-                        galois_mult(0x2, orig_state[1][j]) ^
-                        galois_mult(0x3, orig_state[2][j]) ^
-                        orig_state[3][j];
-                state[2][j] = 
-                        orig_state[0][j] ^
-                        orig_state[1][j] ^
-                        galois_mult(0x2, orig_state[2][j]) ^
-                        galois_mult(0x3, orig_state[3][j]);
-                state[3][j] = 
-                        galois_mult(0x3, orig_state[0][j]) ^
-                        orig_state[1][j] ^
-                        orig_state[2][j] ^
-                        galois_mult(0x2, orig_state[3][j]);
-        }
-}
-
-/**
- * Perform inverted mix columns transformation on state table.
- * 
+        for (size_t j = 0; j < ELEMS; j++) {
+                state[0][j] =
+                    galois_mult(0x2, orig_state[0][j]) ^
+                    galois_mult(0x3, orig_state[1][j]) ^
+                    orig_state[2][j] ^
+                    orig_state[3][j];
+                state[1][j] =
+                    orig_state[0][j] ^
+                    galois_mult(0x2, orig_state[1][j]) ^
+                    galois_mult(0x3, orig_state[2][j]) ^
+                    orig_state[3][j];
+                state[2][j] =
+                    orig_state[0][j] ^
+                    orig_state[1][j] ^
+                    galois_mult(0x2, orig_state[2][j]) ^
+                    galois_mult(0x3, orig_state[3][j]);
+                state[3][j] =
+                    galois_mult(0x3, orig_state[0][j]) ^
+                    orig_state[1][j] ^
+                    orig_state[2][j] ^
+                    galois_mult(0x2, orig_state[3][j]);
+        }
+}
+
+/** Perform inverted mix columns transformation on state table.
+ *
  * @param state State table to be modified.
+ *
  */
 static void inv_mix_columns(uint8_t state[ELEMS][ELEMS])
@@ -341 +343 @@
         memcpy(orig_state, state, BLOCK_LEN);
         
-        for(size_t j = 0; j < ELEMS; j++) {
-                state[0][j] = 
-                        galois_mult(0x0E, orig_state[0][j]) ^
-                        galois_mult(0x0B, orig_state[1][j]) ^
-                        galois_mult(0x0D, orig_state[2][j]) ^
-                        galois_mult(0x09, orig_state[3][j]);
-                state[1][j] = 
-                        galois_mult(0x09, orig_state[0][j]) ^
-                        galois_mult(0x0E, orig_state[1][j]) ^
-                        galois_mult(0x0B, orig_state[2][j]) ^
-                        galois_mult(0x0D, orig_state[3][j]);
-                state[2][j] = 
-                        galois_mult(0x0D, orig_state[0][j]) ^
-                        galois_mult(0x09, orig_state[1][j]) ^
-                        galois_mult(0x0E, orig_state[2][j]) ^
-                        galois_mult(0x0B, orig_state[3][j]);
-                state[3][j] = 
-                        galois_mult(0x0B, orig_state[0][j]) ^
-                        galois_mult(0x0D, orig_state[1][j]) ^
-                        galois_mult(0x09, orig_state[2][j]) ^
-                        galois_mult(0x0E, orig_state[3][j]);
-        }
-}
-
-/**
- * Perform round key transformation on state table.
- * 
- * @param state State table to be modified.
+        for (size_t j = 0; j < ELEMS; j++) {
+                state[0][j] =
+                    galois_mult(0x0e, orig_state[0][j]) ^
+                    galois_mult(0x0b, orig_state[1][j]) ^
+                    galois_mult(0x0d, orig_state[2][j]) ^
+                    galois_mult(0x09, orig_state[3][j]);
+                state[1][j] =
+                    galois_mult(0x09, orig_state[0][j]) ^
+                    galois_mult(0x0e, orig_state[1][j]) ^
+                    galois_mult(0x0b, orig_state[2][j]) ^
+                    galois_mult(0x0d, orig_state[3][j]);
+                state[2][j] =
+                    galois_mult(0x0d, orig_state[0][j]) ^
+                    galois_mult(0x09, orig_state[1][j]) ^
+                    galois_mult(0x0e, orig_state[2][j]) ^
+                    galois_mult(0x0b, orig_state[3][j]);
+                state[3][j] =
+                    galois_mult(0x0b, orig_state[0][j]) ^
+                    galois_mult(0x0d, orig_state[1][j]) ^
+                    galois_mult(0x09, orig_state[2][j]) ^
+                    galois_mult(0x0e, orig_state[3][j]);
+        }
+}
+
+/** Perform round key transformation on state table.
+ *
+ * @param state     State table to be modified.
  * @param round_key Round key to be applied on state table.
+ *
  */
 static void add_round_key(uint8_t state[ELEMS][ELEMS], uint32_t *round_key)
@@ -375 +377 @@
         uint8_t byte_round;
         uint8_t shift;
-        uint32_t mask = 0xFF;
-        
-        for(size_t j = 0; j < ELEMS; j++) {
-                for(size_t i = 0; i < ELEMS; i++) {
-                        shift = 24 - 8*i;
+        uint32_t mask = 0xff;
+        
+        for (size_t j = 0; j < ELEMS; j++) {
+                for (size_t i = 0; i < ELEMS; i++) {
+                        shift = 24 - 8 * i;
                         byte_round = (round_key[j] & (mask << shift)) >> shift;
                         state[i][j] = state[i][j] ^ byte_round;
@@ -386 +388 @@
 }
 
-/**
- * Perform substitution transformation on given word.
- * 
+/** Perform substitution transformation on given word.
+ *
  * @param byte Input word.
- * 
+ *
  * @return Substituted word.
+ *
  */
 static uint32_t sub_word(uint32_t word)
@@ -398 +400 @@
         uint8_t *start = (uint8_t *) &temp;
         
-        for(size_t i = 0; i < 4; i++) {
-                *(start+i) = sub_byte(*(start+i), false);
-        }
+        for (size_t i = 0; i < 4; i++)
+                *(start + i) = sub_byte(*(start + i), false);
         
         return temp;
 }
 
-/**
- * Perform left rotation by one byte on given word.
- * 
+/** Perform left rotation by one byte on given word.
+ *
  * @param byte Input word.
- * 
+ *
  * @return Rotated word.
+ *
  */
 static uint32_t rot_word(uint32_t word)
@@ -417 +418 @@
 }
 
-/**
- * Key expansion procedure for AES algorithm.
- * 
- * @param key Input key.
+/** Key expansion procedure for AES algorithm.
+ *
+ * @param key     Input key.
  * @param key_exp Result key expansion.
+ *
  */
 static void key_expansion(uint8_t *key, uint32_t *key_exp)
@@ -427 +428 @@
         uint32_t temp;
         
-        for(size_t i = 0; i < CIPHER_ELEMS; i++) {
-                key_exp[i] = 
-                        (key[4*i] << 24) + 
-                        (key[4*i+1] << 16) + 
-                        (key[4*i+2] << 8) +
-                        (key[4*i+3]);
-        }
-        
-        for(size_t i = CIPHER_ELEMS; i < ELEMS * (ROUNDS + 1); i++) {
-                temp = key_exp[i-1];
-                
-                if((i % CIPHER_ELEMS) == 0) {
-                        temp = sub_word(rot_word(temp)) ^ 
-                                r_con_array[i/CIPHER_ELEMS - 1];
+        for (size_t i = 0; i < CIPHER_ELEMS; i++) {
+                key_exp[i] =
+                    (key[4 * i] << 24) +
+                    (key[4 * i + 1] << 16) +
+                    (key[4 * i + 2] << 8) +
+                    (key[4 * i + 3]);
+        }
+        
+        for (size_t i = CIPHER_ELEMS; i < ELEMS * (ROUNDS + 1); i++) {
+                temp = key_exp[i - 1];
+                
+                if ((i % CIPHER_ELEMS) == 0) {
+                        temp = sub_word(rot_word(temp)) ^
+                            r_con_array[i / CIPHER_ELEMS - 1];
                 }
                 
@@ -447 +448 @@
 }
 
-/**
- * AES-128 encryption algorithm.
- * 
- * @param key Input key.
- * @param input Input data sequence to be encrypted.
+/** AES-128 encryption algorithm.
+ *
+ * @param key    Input key.
+ * @param input  Input data sequence to be encrypted.
  * @param output Encrypted data sequence.
- * 
- * @return EINVAL when input or key not specified, ENOMEM when pointer for 
- * output is not allocated, otherwise EOK.  
+ *
+ * @return EINVAL when input or key not specified,
+ *         ENOMEM when pointer for output is not allocated,
+ *         otherwise EOK.
+ *
  */
 int aes_encrypt(uint8_t *key, uint8_t *input, uint8_t *output)
 {
-        if(!key || !input)
+        if ((!key) || (!input))
                 return EINVAL;
         
-        if(!output)
+        if (!output)
                 return ENOMEM;
         
         /* Create key expansion. */
-        uint32_t key_exp[ELEMS * (ROUNDS+1)];
+        uint32_t key_exp[ELEMS * (ROUNDS + 1)];
         key_expansion(key, key_exp);
         
         /* Copy input into state array. */
         uint8_t state[ELEMS][ELEMS];
-        for(size_t i = 0; i < ELEMS; i++) {
-                for(size_t j = 0; j < ELEMS; j++) {
-                        state[i][j] = input[i + ELEMS*j];
-                }
+        for (size_t i = 0; i < ELEMS; i++) {
+                for (size_t j = 0; j < ELEMS; j++)
+                        state[i][j] = input[i + ELEMS * j];
         }
         
@@ -480 +481 @@
         add_round_key(state, key_exp);
         
-        for(size_t k = 1; k <= ROUNDS; k++) {
+        for (size_t k = 1; k <= ROUNDS; k++) {
                 sub_bytes(state, false);
                 shift_rows(state);
-                if(k < ROUNDS)
+                
+                if (k < ROUNDS)
                         mix_columns(state);
-                add_round_key(state, key_exp + k*ELEMS);
+                
+                add_round_key(state, key_exp + k * ELEMS);
         }
         
         /* Copy state array into output. */
-        for(size_t i = 0; i < ELEMS; i++) {
-                for(size_t j = 0; j < ELEMS; j++) {
-                        output[i + j*ELEMS] = state[i][j];
-                }
+        for (size_t i = 0; i < ELEMS; i++) {
+                for (size_t j = 0; j < ELEMS; j++)
+                        output[i + j * ELEMS] = state[i][j];
         }
         
@@ -498 +500 @@
 }
 
-/**
- * AES-128 decryption algorithm.
- * 
- * @param key Input key.
- * @param input Input data sequence to be decrypted.
+/** AES-128 decryption algorithm.
+ *
+ * @param key    Input key.
+ * @param input  Input data sequence to be decrypted.
  * @param output Decrypted data sequence.
- * 
- * @return EINVAL when input or key not specified, ENOMEM when pointer for 
- * output is not allocated, otherwise EOK.  
+ *
+ * @return EINVAL when input or key not specified,
+ *         ENOMEM when pointer for output is not allocated,
+ *         otherwise EOK.
+ *
  */
 int aes_decrypt(uint8_t *key, uint8_t *input, uint8_t *output)
 {
-        if(!key || !input)
+        if ((!key) || (!input))
                 return EINVAL;
         
-        if(!output)
+        if (!output)
                 return ENOMEM;
         
         /* Create key expansion. */
-        uint32_t key_exp[ELEMS * (ROUNDS+1)];
+        uint32_t key_exp[ELEMS * (ROUNDS + 1)];
         key_expansion(key, key_exp);
         
         /* Copy input into state array. */
         uint8_t state[ELEMS][ELEMS];
-        for(size_t i = 0; i < ELEMS; i++) {
-                for(size_t j = 0; j < ELEMS; j++) {
-                        state[i][j] = input[i + ELEMS*j];
-                }
+        for (size_t i = 0; i < ELEMS; i++) {
+                for (size_t j = 0; j < ELEMS; j++)
+                        state[i][j] = input[i + ELEMS * j];
         }
         
         /* Processing loop. */
-        add_round_key(state, key_exp + ROUNDS*ELEMS);
-        
-        for(int k = ROUNDS - 1; k >= 0; k--) {
+        add_round_key(state, key_exp + ROUNDS * ELEMS);
+        
+        for (int k = ROUNDS - 1; k >= 0; k--) {
                 inv_shift_rows(state);
                 sub_bytes(state, true);
-                add_round_key(state, key_exp + k*ELEMS);
-                if(k > 0)
+                add_round_key(state, key_exp + k * ELEMS);
+                
+                if (k > 0)
                         inv_mix_columns(state);
         }
         
         /* Copy state array into output. */
-        for(size_t i = 0; i < ELEMS; i++) {
-                for(size_t j = 0; j < ELEMS; j++) {
-                        output[i + j*ELEMS] = state[i][j];
-                }
+        for (size_t i = 0; i < ELEMS; i++) {
+                for (size_t j = 0; j < ELEMS; j++)
+                        output[i + j * ELEMS] = state[i][j];
         }
         

uspace/lib/crypto/crypto.c

@@ -28 +28 @@
 
 /** @file crypto.c
- * 
+ *
  * Cryptographic functions library.
  */
@@ -37 +37 @@
 #include <errno.h>
 #include <byteorder.h>
-
 #include "crypto.h"
 
-/* Hash function procedure definition. */
-typedef void (*HASH_FUNC)(uint32_t*, uint32_t*);
-
-/* Length of HMAC block. */
-#define HMAC_BLOCK_LENGTH 64
-
-/* Ceiling for UINT32. */
-#define ceil_uint32(val) (((val) - (uint32_t)(val)) > 0 ? \
-        (uint32_t)((val) + 1) : (uint32_t)(val))
-
-/* Floor for UINT32. */
-#define floor_uint32(val) (((val) - (uint32_t)(val)) < 0 ? \
-        (uint32_t)((val) - 1) : (uint32_t)(val))
-
-/* Pick value at specified index from array or zero if out of bounds. */
-#define get_at(input, size, i) (i < size ? input[i] : 0)
-
-/* Init values used in SHA1 and MD5 functions. */
+/** Hash function procedure definition. */
+typedef void (*hash_fnc_t)(uint32_t *, uint32_t *);
+
+/** Length of HMAC block. */
+#define HMAC_BLOCK_LENGTH  64
+
+/** Ceiling for uint32_t. */
+#define ceil_uint32(val) \
+        (((val) - (uint32_t) (val)) > 0 ? \
+        (uint32_t) ((val) + 1) : (uint32_t) (val))
+
+/** Floor for uint32_t. */
+#define floor_uint32(val) \
+        (((val) - (uint32_t) (val)) < 0 ? \
+        (uint32_t) ((val) - 1) : (uint32_t) (val))
+
+/** Pick value at specified index from array or zero if out of bounds. */
+#define get_at(input, size, i) \
+        ((i) < (size) ? (input[i]) : 0)
+
+/** Init values used in SHA1 and MD5 functions. */
 static const uint32_t hash_init[] = {
-        0x67452301, 0xEFCDAB89, 0x98BADCFE, 0x10325476, 0xC3D2E1F0
+        0x67452301, 0xefcdab89, 0x98badcfe, 0x10325476, 0xc3d2e1f0
 };
 
-/* Shift amount array for MD5 algorithm. */
+/** Shift amount array for MD5 algorithm. */
 static const uint32_t md5_shift[] = {
         7, 12, 17, 22,  7, 12, 17, 22,  7, 12, 17, 22,  7, 12, 17, 22,
@@ -70 +72 @@
 };
 
-/* Substitution box for MD5 algorithm. */
+/** Substitution box for MD5 algorithm. */
 static const uint32_t md5_sbox[] = {
         0xd76aa478, 0xe8c7b756, 0x242070db, 0xc1bdceee,
@@ -90 +92 @@
 };
 
-/**
- * Working procedure of MD5 cryptographic hash function.
- * 
- * @param h Working array with interim hash parts values.
+/** Working procedure of MD5 cryptographic hash function.
+ *
+ * @param h         Working array with interim hash parts values.
  * @param sched_arr Input array with scheduled values from input string.
+ *
  */
 static void md5_proc(uint32_t *h, uint32_t *sched_arr)
 {
         uint32_t f, g, temp;
-        uint32_t w[HASH_MD5/4];
-        
-        memcpy(w, h, (HASH_MD5/4) * sizeof(uint32_t));
-        
-        for(size_t k = 0; k < 64; k++) {
-                if(k < 16) {
+        uint32_t w[HASH_MD5 / 4];
+        
+        memcpy(w, h, (HASH_MD5 / 4) * sizeof(uint32_t));
+        
+        for (size_t k = 0; k < 64; k++) {
+                if (k < 16) {
                         f = (w[1] & w[2]) | (~w[1] & w[3]);
                         g = k;
-                } else if(k >= 16 && k < 32) {
+                } else if ((k >= 16) && (k < 32)) {
                         f = (w[1] & w[3]) | (w[2] & ~w[3]);
-                        g = (5*k + 1) % 16;
-                } else if(k >= 32 && k < 48) {
+                        g = (5 * k + 1) % 16;
+                } else if ((k >= 32) && (k < 48)) {
                         f = w[1] ^ w[2] ^ w[3];
-                        g = (3*k + 5) % 16;
+                        g = (3 * k + 5) % 16;
                 } else {
                         f = w[2] ^ (w[1] | ~w[3]);
-                        g = 7*k % 16;
+                        g = 7 * k % 16;
                 }
+                
                 temp = w[3];
                 w[3] = w[2];
                 w[2] = w[1];
-                w[1] += rotl_uint32(w[0] + f + md5_sbox[k] + 
-                        uint32_t_byteorder_swap(sched_arr[g]),  
-                        md5_shift[k]);
+                w[1] += rotl_uint32(w[0] + f + md5_sbox[k] +
+                    uint32_t_byteorder_swap(sched_arr[g]),
+                    md5_shift[k]);
                 w[0] = temp;
         }
         
-        for(uint8_t k = 0; k < HASH_MD5/4; k++)
+        for (uint8_t k = 0; k < HASH_MD5 / 4; k++)
                 h[k] += w[k];
 }
 
-/**
- * Working procedure of SHA-1 cryptographic hash function.
- * 
- * @param h Working array with interim hash parts values.
+/** Working procedure of SHA-1 cryptographic hash function.
+ *
+ * @param h         Working array with interim hash parts values.
  * @param sched_arr Input array with scheduled values from input string.
+ *
  */
 static void sha1_proc(uint32_t *h, uint32_t *sched_arr)
 {
         uint32_t f, cf, temp;
-        uint32_t w[HASH_SHA1/4];
-        
-        for(size_t k = 16; k < 80; k++) {
+        uint32_t w[HASH_SHA1 / 4];
+        
+        for (size_t k = 16; k < 80; k++) {
                 sched_arr[k] = rotl_uint32(
-                        sched_arr[k-3] ^
-                        sched_arr[k-8] ^
-                        sched_arr[k-14] ^
-                        sched_arr[k-16], 
-                        1);
-        }
-
-        memcpy(w, h, (HASH_SHA1/4) * sizeof(uint32_t));
-        
-        for(size_t k = 0; k < 80; k++) {
-                if(k < 20) {
+                    sched_arr[k-3] ^
+                    sched_arr[k-8] ^
+                    sched_arr[k-14] ^
+                    sched_arr[k-16],
+                    1);
+        }
+        
+        memcpy(w, h, (HASH_SHA1 / 4) * sizeof(uint32_t));
+        
+        for (size_t k = 0; k < 80; k++) {
+                if (k < 20) {
                         f = (w[1] & w[2]) | (~w[1] & w[3]);
                         cf = 0x5A827999;
-                } else if(k >= 20 && k < 40) {
+                } else if ((k >= 20) && (k < 40)) {
                         f = w[1] ^ w[2] ^ w[3];
-                        cf = 0x6ED9EBA1;
-                } else if(k >= 40 && k < 60) {
+                        cf = 0x6ed9eba1;
+                } else if ((k >= 40) && (k < 60)) {
                         f = (w[1] & w[2]) | (w[1] & w[3]) | (w[2] & w[3]);
-                        cf = 0x8F1BBCDC;
+                        cf = 0x8f1bbcdc;
                 } else {
                         f = w[1] ^ w[2] ^ w[3];
-                        cf = 0xCA62C1D6;
+                        cf = 0xca62c1d6;
                 }
-
+                
                 temp = rotl_uint32(w[0], 5) + f + w[4] + cf + sched_arr[k];
-
+                
                 w[4] = w[3];
                 w[3] = w[2];
@@ -175 +178 @@
                 w[0] = temp;
         }
-
-        for(uint8_t k = 0; k < HASH_SHA1/4; k++)
+        
+        for (uint8_t k = 0; k < HASH_SHA1 / 4; k++)
                 h[k] += w[k];
 }
 
-/**
- * Create hash based on selected algorithm.
- * 
- * @param input Input message byte sequence.
+/** Create hash based on selected algorithm.
+ *
+ * @param input      Input message byte sequence.
  * @param input_size Size of message sequence.
- * @param output Result hash byte sequence.
- * @param hash_sel Hash function selector.
- * 
- * @return EINVAL when input not specified, ENOMEM when pointer for 
- * output hash result is not allocated, otherwise EOK. 
+ * @param output     Result hash byte sequence.
+ * @param hash_sel   Hash function selector.
+ *
+ * @return EINVAL when input not specified,
+ *         ENOMEM when pointer for output hash result
+ *         is not allocated, otherwise EOK.
+ *
  */
 int create_hash(uint8_t *input, size_t input_size, uint8_t *output,
-        hash_func_t hash_sel)
-{
-        if(!input)
+    hash_func_t hash_sel)
+{
+        if (!input)
                 return EINVAL;
         
-        if(!output)
+        if (!output)
                 return ENOMEM;
         
-        HASH_FUNC hash_func = (hash_sel == HASH_MD5) ? md5_proc : sha1_proc;
+        hash_fnc_t hash_func = (hash_sel == HASH_MD5) ? md5_proc : sha1_proc;
         
         /* Prepare scheduled input. */
@@ -207 +211 @@
         work_input[input_size] = 0x80;
         
-        size_t blocks = ceil_uint32((((double)input_size + 1) / 4 + 2) / 16);
+        // FIXME: double?
+        size_t blocks = ceil_uint32((((double) input_size + 1) / 4 + 2) / 16);
         uint32_t work_arr[blocks * 16];
-        for(size_t i = 0; i < blocks; i++) {
-                for(size_t j = 0; j < 16; j++) {
-                        work_arr[i*16 + j] = 
-                        (get_at(work_input, input_size+1, i*64+j*4) << 24) |
-                        (get_at(work_input, input_size+1, i*64+j*4+1) << 16) |
-                        (get_at(work_input, input_size+1, i*64+j*4+2) << 8) |
-                        get_at(work_input, input_size+1, i*64+j*4+3);
+        for (size_t i = 0; i < blocks; i++) {
+                for (size_t j = 0; j < 16; j++) {
+                        work_arr[i*16 + j] =
+                            (get_at(work_input, input_size + 1, i * 64 + j * 4) << 24) |
+                            (get_at(work_input, input_size + 1, i * 64 + j * 4 + 1) << 16) |
+                            (get_at(work_input, input_size + 1, i * 64 + j * 4 + 2) << 8) |
+                            get_at(work_input, input_size + 1, i * 64 + j * 4 + 3);
                 }
         }
         
-        uint64_t bits_size = (uint64_t)(input_size * 8);
-        if(hash_sel == HASH_MD5)
+        uint64_t bits_size = (uint64_t) (input_size * 8);
+        if (hash_sel == HASH_MD5)
                 bits_size = uint64_t_byteorder_swap(bits_size);
         
         work_arr[(blocks - 1) * 16 + 14] = bits_size >> 32;
-        work_arr[(blocks - 1) * 16 + 15] = bits_size & 0xFFFFFFFF;
+        work_arr[(blocks - 1) * 16 + 15] = bits_size & 0xffffffff;
         
         /* Hash computation. */
-        uint32_t h[hash_sel/4];
-        memcpy(h, hash_init, (hash_sel/4) * sizeof(uint32_t));
+        uint32_t h[hash_sel / 4];
+        memcpy(h, hash_init, (hash_sel / 4) * sizeof(uint32_t));
         uint32_t sched_arr[80];
-        for(size_t i = 0; i < blocks; i++) {
-                for(size_t k = 0; k < 16; k++) {
-                        sched_arr[k] = work_arr[i*16 + k];
-                }
+        for (size_t i = 0; i < blocks; i++) {
+                for (size_t k = 0; k < 16; k++)
+                        sched_arr[k] = work_arr[i * 16 + k];
                 
                 hash_func(h, sched_arr);
@@ -239 +243 @@
         
         /* Copy hash parts into final result. */
-        for(size_t i = 0; i < hash_sel/4; i++) {
-                if(hash_sel == HASH_SHA1)
+        for (size_t i = 0; i < hash_sel / 4; i++) {
+                if (hash_sel == HASH_SHA1)
                         h[i] = uint32_t_byteorder_swap(h[i]);
-                memcpy(output + i*sizeof(uint32_t), &h[i], sizeof(uint32_t));
+                
+                memcpy(output + i * sizeof(uint32_t), &h[i], sizeof(uint32_t));
         }
         
@@ -248 +253 @@
 }
 
-/**
- * Hash-based message authentication code.
- * 
- * @param key Cryptographic key sequence.
+/** Hash-based message authentication code.
+ *
+ * @param key      Cryptographic key sequence.
  * @param key_size Size of key sequence.
- * @param msg Message sequence.
+ * @param msg      Message sequence.
  * @param msg_size Size of message sequence.
- * @param hash Output parameter for result hash.
+ * @param hash     Output parameter for result hash.
  * @param hash_sel Hash function selector.
- * 
- * @return EINVAL when key or message not specified, ENOMEM when pointer for 
- * output hash result is not allocated, otherwise EOK.
+ *
+ * @return EINVAL when key or message not specified,
+ *         ENOMEM when pointer for output hash result
+ *         is not allocated, otherwise EOK.
+ *
  */
 int hmac(uint8_t *key, size_t key_size, uint8_t *msg, size_t msg_size, 
-        uint8_t *hash, hash_func_t hash_sel)
-{
-        if(!key || !msg)
+    uint8_t *hash, hash_func_t hash_sel)
+{
+        if ((!key) || (!msg))
                 return EINVAL;
         
-        if(!hash)
+        if (!hash)
                 return ENOMEM;
         
@@ -276 +282 @@
         memset(work_key, 0, HMAC_BLOCK_LENGTH);
         
-        if(key_size > HMAC_BLOCK_LENGTH) {
+        if(key_size > HMAC_BLOCK_LENGTH)
                 create_hash(key, key_size, work_key, hash_sel);
-        } else {
+        else
                 memcpy(work_key, key, key_size);
-        }
-        
-        for(size_t i = 0; i < HMAC_BLOCK_LENGTH; i++) {
-                o_key_pad[i] = work_key[i] ^ 0x5C;
+        
+        for (size_t i = 0; i < HMAC_BLOCK_LENGTH; i++) {
+                o_key_pad[i] = work_key[i] ^ 0x5c;
                 i_key_pad[i] = work_key[i] ^ 0x36;
         }
@@ -291 +296 @@
         memcpy(temp_work + HMAC_BLOCK_LENGTH, msg, msg_size);
         
-        create_hash(temp_work, HMAC_BLOCK_LENGTH + msg_size, temp_hash, 
-                hash_sel);
+        create_hash(temp_work, HMAC_BLOCK_LENGTH + msg_size, temp_hash,
+            hash_sel);
         
         memcpy(temp_work, o_key_pad, HMAC_BLOCK_LENGTH);
@@ -302 +307 @@
 }
 
-/**
- * Password-Based Key Derivation Function 2 as defined in RFC 2898,
- * using HMAC-SHA1 with 4096 iterations and 32 bytes key result used
- * for WPA/WPA2.
- * 
- * @param pass Password sequence.
+/** Password-Based Key Derivation Function 2.
+ *
+ * As defined in RFC 2898, using HMAC-SHA1 with 4096 iterations
+ * and 32 bytes key result used for WPA/WPA2.
+ *
+ * @param pass      Password sequence.
  * @param pass_size Password sequence length.
- * @param salt Salt sequence to be used with password.
+ * @param salt      Salt sequence to be used with password.
  * @param salt_size Salt sequence length.
- * @param hash Output parameter for result hash (32 byte value).
- * 
- * @return EINVAL when pass or salt not specified, ENOMEM when pointer for 
- * output hash result is not allocated, otherwise EOK.
- */
-int pbkdf2(uint8_t *pass, size_t pass_size, uint8_t *salt, size_t salt_size, 
-        uint8_t *hash)
-{
-        if(!pass || !salt)
+ * @param hash      Output parameter for result hash (32 byte value).
+ *
+ * @return EINVAL when pass or salt not specified,
+ *         ENOMEM when pointer for output hash result
+ *         is not allocated, otherwise EOK.
+ *
+ */
+int pbkdf2(uint8_t *pass, size_t pass_size, uint8_t *salt, size_t salt_size,
+    uint8_t *hash)
+{
+        if ((!pass) || (!salt))
                 return EINVAL;
         
-        if(!hash)
+        if (!hash)
                 return ENOMEM;
         
@@ -330 +337 @@
         uint8_t temp_hmac[HASH_SHA1];
         uint8_t xor_hmac[HASH_SHA1];
-        uint8_t temp_hash[HASH_SHA1*2];
-        
-        for(size_t i = 0; i < 2; i++) {
-                uint32_t be_i = host2uint32_t_be(i+1);
+        uint8_t temp_hash[HASH_SHA1 * 2];
+        
+        for (size_t i = 0; i < 2; i++) {
+                uint32_t be_i = host2uint32_t_be(i + 1);
+                
                 memcpy(work_salt + salt_size, &be_i, 4);
                 hmac(pass, pass_size, work_salt, salt_size + 4,
-                        work_hmac, HASH_SHA1);
+                    work_hmac, HASH_SHA1);
                 memcpy(xor_hmac, work_hmac, HASH_SHA1);
                 
-                for(size_t k = 1; k < 4096; k++) {
+                for (size_t k = 1; k < 4096; k++) {
                         memcpy(temp_hmac, work_hmac, HASH_SHA1);
-                        hmac(pass, pass_size, temp_hmac, HASH_SHA1, 
-                                work_hmac, HASH_SHA1);
-                        for(size_t t = 0; t < HASH_SHA1; t++) {
+                        hmac(pass, pass_size, temp_hmac, HASH_SHA1,
+                            work_hmac, HASH_SHA1);
+                        
+                        for (size_t t = 0; t < HASH_SHA1; t++)
                                 xor_hmac[t] ^= work_hmac[t];
-                        }
                 }
-                memcpy(temp_hash + i*HASH_SHA1, xor_hmac, HASH_SHA1);
+                
+                memcpy(temp_hash + i * HASH_SHA1, xor_hmac, HASH_SHA1);
         }
         

uspace/lib/crypto/crypto.h

@@ -32 +32 @@
 #include <sys/types.h>
 
-#define AES_CIPHER_LENGTH 16
-#define PBKDF2_KEY_LENGTH 32
+#define AES_CIPHER_LENGTH  16
+#define PBKDF2_KEY_LENGTH  32
 
-/* Left rotation for UINT32. */
-#define rotl_uint32(val, shift) (((val) << shift) | ((val) >> (32 - shift)))
+/* Left rotation for uint32_t. */
+#define rotl_uint32(val, shift) \
+        (((val) << shift) | ((val) >> (32 - shift)))
 
-/* Right rotation for UINT32. */
-#define rotr_uint32(val, shift) (((val) >> shift) | ((val) << (32 - shift)))
+/* Right rotation for uint32_t. */
+#define rotr_uint32(val, shift) \
+        (((val) >> shift) | ((val) << (32 - shift)))
 
 /** Hash function selector and also result hash length indicator. */
 typedef enum {
-        HASH_MD5 =      16,
-        HASH_SHA1 =     20
+        HASH_MD5 =  16,
+        HASH_SHA1 = 20
 } hash_func_t;
 
-extern int rc4(uint8_t *key, size_t key_size, uint8_t *input, size_t input_size, 
-        size_t skip, uint8_t *output);
-extern int aes_encrypt(uint8_t *key, uint8_t *input, uint8_t *output);
-extern int aes_decrypt(uint8_t *key, uint8_t *input, uint8_t *output);
-extern int create_hash(uint8_t *input, size_t input_size, uint8_t *output, 
-        hash_func_t hash_sel);
-extern int hmac(uint8_t *key, size_t key_size, uint8_t *msg, size_t msg_size, 
-        uint8_t *hash, hash_func_t hash_sel);
-extern int pbkdf2(uint8_t *pass, size_t pass_size, uint8_t *salt, 
-        size_t salt_size, uint8_t *hash);
+extern int rc4(uint8_t *, size_t, uint8_t *, size_t, size_t, uint8_t *);
+extern int aes_encrypt(uint8_t *, uint8_t *, uint8_t *);
+extern int aes_decrypt(uint8_t *, uint8_t *, uint8_t *);
+extern int create_hash(uint8_t *, size_t, uint8_t *, hash_func_t);
+extern int hmac(uint8_t *, size_t, uint8_t *, size_t, uint8_t *, hash_func_t);
+extern int pbkdf2(uint8_t *, size_t, uint8_t *, size_t, uint8_t *);
 
 #endif

uspace/lib/crypto/rc4.c

@@ -28 +28 @@
 
 /** @file rc4.c
- * 
+ *
  * Implementation of ARC4 symmetric cipher cryptographic algorithm.
- * 
+ *
  */
 
 #include <errno.h>
 #include <mem.h>
-
 #include "crypto.h"
 
 /* Sbox table size. */
-#define SBOX_SIZE 256
+#define SBOX_SIZE  256
 
-/**
- * Swap two values in sbox.
- * 
- * @param i First index of value in sbox to be swapped.
- * @param j Second index of value in sbox to be swapped.
+/** Swap two values in sbox.
+ *
+ * @param i    First index of value in sbox to be swapped.
+ * @param j    Second index of value in sbox to be swapped.
  * @param sbox Sbox to be modified.
+ *
  */
 static void swap(size_t i, size_t j, uint8_t *sbox)
@@ -55 +54 @@
 }
 
-/**
- * Sbox initialization procedure.
- * 
- * @param key Input key.
+/** Sbox initialization procedure.
+ *
+ * @param key      Input key.
  * @param key_size Size of key sequence.
- * @param sbox Place for result sbox.
+ * @param sbox     Place for result sbox.
+ *
  */
 static void create_sbox(uint8_t *key, size_t key_size, uint8_t *sbox)
 {
-        for(size_t i = 0; i < SBOX_SIZE; i++) {
+        for (size_t i = 0; i < SBOX_SIZE; i++)
                 sbox[i] = i;
-        }
         
         uint8_t j = 0;
-        for(size_t i = 0; i < SBOX_SIZE; i++) {
+        for (size_t i = 0; i < SBOX_SIZE; i++) {
                 j = j + sbox[i] + key[i % key_size];
                 swap(i, j, sbox);
@@ -75 +73 @@
 }
 
-/**
- * ARC4 encryption/decryption algorithm.
- * 
- * @param key Input key.
- * @param key_size Size of key sequence.
- * @param input Input data sequence to be processed.
+/** ARC4 encryption/decryption algorithm.
+ *
+ * @param key        Input key.
+ * @param key_size   Size of key sequence.
+ * @param input      Input data sequence to be processed.
  * @param input_size Size of input data sequence.
- * @param skip Number of bytes to be skipped from the beginning of key stream.
- * @param output Result data sequence.
- * 
- * @return EINVAL when input or key not specified, ENOMEM when pointer for 
- * output is not allocated, otherwise EOK.  
+ * @param skip       Number of bytes to be skipped from
+ *                   the beginning of key stream.
+ * @param output     Result data sequence.
+ *
+ * @return EINVAL when input or key not specified,
+ *         ENOMEM when pointer for output is not allocated,
+ *         otherwise EOK.
+ *
  */
-int rc4(uint8_t *key, size_t key_size, uint8_t *input, size_t input_size, 
-        size_t skip, uint8_t *output)
+int rc4(uint8_t *key, size_t key_size, uint8_t *input, size_t input_size,
+    size_t skip, uint8_t *output)
 {
-        if(!key || !input)
+        if ((!key) || (!input))
                 return EINVAL;
         
-        if(!output)
+        if (!output)
                 return ENOMEM;
         
@@ -102 +102 @@
         
         /* Skip first x bytes. */
-        uint8_t i = 0, j = 0;
-        for(size_t k = 0; k < skip; k++) {
+        uint8_t i = 0;
+        uint8_t j = 0;
+        for (size_t k = 0; k < skip; k++) {
                 i = i + 1;
                 j = j + sbox[i];
@@ -111 +112 @@
         /* Processing loop. */
         uint8_t val;
-        for(size_t k = 0; k < input_size; k++) {
+        for (size_t k = 0; k < input_size; k++) {
                 i = i + 1;
                 j = j + sbox[i];

uspace/lib/drv/generic/remote_ieee80211.c

@@ -40 +40 @@
 #include <inet/dhcp.h>
 #include <inet/inetcfg.h>
-
 #include "ops/ieee80211.h"
 #include "ieee80211_iface.h"
 #include "nic_iface.h"
 
-#define MAX_STRING_SIZE 32
+#define MAX_STRING_SIZE  32
 
 /** IEEE 802.11 RPC functions IDs. */
@@ -56 +55 @@
 /** Get scan results from IEEE 802.11 device
  *
- * @param[in] dev_sess Device session.
- * @param[out] results Structure where to put scan results.
+ * @param[in]  dev_sess Device session.
+ * @param[out] results  Structure where to put scan results.
  *
  * @return EOK If the operation was successfully completed,
- * negative error code otherwise.
- */
-int ieee80211_get_scan_results(async_sess_t *dev_sess, 
-        ieee80211_scan_results_t *results, bool now)
+ *         negative error code otherwise.
+ *
+ */
+int ieee80211_get_scan_results(async_sess_t *dev_sess,
+    ieee80211_scan_results_t *results, bool now)
 {
         assert(results);
@@ -71 +71 @@
         aid_t aid = async_send_2(exch, DEV_IFACE_ID(IEEE80211_DEV_IFACE),
             IEEE80211_GET_SCAN_RESULTS, now, NULL);
-        int rc = async_data_read_start(exch, results, sizeof(ieee80211_scan_results_t));
+        int rc = async_data_read_start(exch, results,
+            sizeof(ieee80211_scan_results_t));
         async_exchange_end(exch);
-
+        
         sysarg_t res;
         async_wait_for(aid, &res);
@@ -79 +80 @@
         if(res != EOK)
                 return (int) res;
-        else
-                return rc;
+        
+        return rc;
 }
 
 static bool mac_matches(uint8_t *mac1, uint8_t *mac2)
 {
-        for(size_t i = 0; i < ETH_ADDR; i++) {
-                if(mac1[i] != mac2[i])
+        for (size_t i = 0; i < ETH_ADDR; i++) {
+                if (mac1[i] != mac2[i])
                         return false;
         }
@@ -108 +109 @@
                         return -1;
                 
-                if(mac_matches(mac, link_info.mac_addr)) {
+                if (mac_matches(mac, link_info.mac_addr))
                         return link_list[i];
-                }
         }
         
@@ -118 +118 @@
 /** Connect to specified network.
  *
- * @param[in] dev_sess Device session.
+ * @param[in] dev_sess   Device session.
  * @param[in] ssid_start Network SSID prefix.
- * @param[in] password Network password (pass empty string if not needed).
+ * @param[in] password   Network password (pass empty string if not needed).
  *
  * @return EOK If the operation was successfully completed,
- * negative error code otherwise.
+ *         negative error code otherwise.
+ *
  */
 int ieee80211_connect(async_sess_t *dev_sess, char *ssid_start, char *password)
@@ -136 +137 @@
             IEEE80211_CONNECT, NULL);
         
-        sysarg_t rc = async_data_write_start(exch, ssid_start, 
-                str_size(ssid_start) + 1);
+        sysarg_t rc = async_data_write_start(exch, ssid_start,
+            str_size(ssid_start) + 1);
         if (rc != EOK) {
                 async_exchange_end(exch);
@@ -144 +145 @@
                 if (rc_orig == EOK)
                         return (int) rc;
-                else
-                        return (int) rc_orig;
-        }
-        
-        if(password == NULL) {
+                
+                return (int) rc_orig;
+        }
+        
+        // FIXME: Typecasting string literal
+        if (password == NULL)
                 password = (char *) "";
-        }
         
         rc = async_data_write_start(exch, password, str_size(password) + 1);
@@ -159 +160 @@
                 if (rc_orig == EOK)
                         return (int) rc;
-                else
-                        return (int) rc_orig;
+                
+                return (int) rc_orig;
         }
         
         async_exchange_end(exch);
-
+        
         async_wait_for(aid, &rc);
         if (rc != EOK)
@@ -176 +177 @@
         
         sysarg_t link_id = get_link_id(wifi_mac.address);
-        if(link_id == ((sysarg_t) -1))
+        if (link_id == ((sysarg_t) -1))
                 return EINVAL;
         
@@ -189 +190 @@
  *
  * @return EOK If the operation was successfully completed,
- * negative error code otherwise.
+ *         negative error code otherwise.
+ *
  */
 int ieee80211_disconnect(async_sess_t *dev_sess)
@@ -198 +200 @@
         async_exchange_end(exch);
         
-        if(rc != EOK)
+        if (rc != EOK)
                 return rc;
         
@@ -227 +229 @@
                         return rc;
                 
-                if(mac_matches(wifi_mac.address, link_info.mac_addr)) {
-                        if(str_test_prefix(addr_info.name, "dhcp")) {
+                if (mac_matches(wifi_mac.address, link_info.mac_addr)) {
+                        if (str_test_prefix(addr_info.name, "dhcp")) {
                                 rc = inetcfg_addr_delete(addr_list[i]);
-                                if(rc != EOK)
+                                if (rc != EOK)
                                         return rc;
+                                
                                 break;
                         }
@@ -237 +240 @@
         }
         
-        /* 
+        /*
          * TODO: At this moment there can be only one DHCP route,
          * so it must be reimplemented after this limitation will be
@@ -252 +255 @@
                         return rc;
                 
-                if(str_test_prefix(route_info.name, "dhcp")) {
+                if (str_test_prefix(route_info.name, "dhcp")) {
                         rc = inetcfg_sroute_delete(route_list[i]);
-                        if(rc != EOK)
+                        if (rc != EOK)
                                 return rc;
+                        
                         break;
                 }

uspace/lib/drv/include/ieee80211_iface.h

@@ -27 +27 @@
  */
 
- /** @addtogroup libc
+/** @addtogroup libc
  * @{
  */
@@ -39 +39 @@
 #include <async.h>
 
-extern int ieee80211_get_scan_results(async_sess_t *, 
-        ieee80211_scan_results_t *, bool);
+extern int ieee80211_get_scan_results(async_sess_t *,
+    ieee80211_scan_results_t *, bool);
 extern int ieee80211_connect(async_sess_t *, char *, char *);
 extern int ieee80211_disconnect(async_sess_t *);

uspace/lib/drv/include/ops/ieee80211.h

@@ -42 +42 @@
 /** IEEE 802.11 interface functions definition. */
 typedef struct ieee80211_iface {
-        /**
-         * Fetch scan results from IEEE 802.11 device.
-         * 
-         * @param fun IEEE 802.11 function.
+        /** Fetch scan results from IEEE 802.11 device.
+         *
+         * @param fun     IEEE 802.11 function.
          * @param results Structure where to put scan results.
-         * @param now Whether to initiate scan immediately.
-         * 
+         * @param now     Whether to initiate scan immediately.
+         *
          * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*get_scan_results)(ddf_fun_t *, ieee80211_scan_results_t *, bool);
         
-        /** 
-         * Connect IEEE 802.11 device to specified network.
+        /** Connect IEEE 802.11 device to specified network.
          *
-         * @param fun IEEE 802.11 function.
-         * @param ssid Network SSID.
+         * @param fun      IEEE 802.11 function.
+         * @param ssid     Network SSID.
          * @param password Network password (empty string if not needed).
-         * 
+         *
          * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*connect)(ddf_fun_t *, char *, char *);
         
-        /** 
-         * Disconnect IEEE 802.11 device from network.
+        /** Disconnect IEEE 802.11 device from network.
          *
          * @param fun IEEE 802.11 function.
-         * 
+         *
          * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*disconnect)(ddf_fun_t *);

uspace/lib/ieee80211/Makefile

@@ -45 +45 @@
         src/ieee80211_impl.c \
         src/ieee80211_iface_impl.c \
-        
 
 include $(USPACE_PREFIX)/Makefile.common

uspace/lib/ieee80211/include/ieee80211.h

@@ -27 +27 @@
  */
 
-/** 
+/**
  * @addtogroup libieee80211
  * @{
@@ -43 +43 @@
 #include <ops/ieee80211.h>
 
-#define DEVICE_CATEGORY_IEEE80211 "ieee80211"
+#define DEVICE_CATEGORY_IEEE80211  "ieee80211"
 
 struct ieee80211_dev;
@@ -49 +49 @@
 
 /** Initial channel frequency. */
-#define IEEE80211_FIRST_FREQ 2412
+#define IEEE80211_FIRST_FREQ  2412
 
 /** Max supported channel frequency. */
-#define IEEE80211_MAX_FREQ 2472
-
-/* Gap between IEEE80211 channels in MHz. */
-#define IEEE80211_CHANNEL_GAP 5
-
-/* Max AMPDU factor. */
-#define IEEE80211_MAX_AMPDU_FACTOR 13
-
-/* Max authentication password length. */
-#define IEEE80211_MAX_PASSW_LEN 64
+#define IEEE80211_MAX_FREQ  2472
+
+/** Gap between IEEE80211 channels in MHz. */
+#define IEEE80211_CHANNEL_GAP  5
+
+/** Max AMPDU factor. */
+#define IEEE80211_MAX_AMPDU_FACTOR  13
+
+/** Max authentication password length. */
+#define IEEE80211_MAX_PASSW_LEN  64
 
 /** IEEE 802.11 b/g supported data rates in units of 500 kb/s. */
@@ -97 +97 @@
 /** IEEE 802.11 callback functions. */
 typedef struct {
-        /**
-         * Function that is called at device initalization. This should
-         * get device into running state.
-         * 
-         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
-         * 
-         * @return EOK if succeed, negative error code otherwise.
+        /** unction that is called at device initalization.
+         *
+         * This should get device into running state.
+         *
+         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
+         *
+         * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*start)(struct ieee80211_dev *);
         
-        /**
-         * Scan neighborhood for networks. There should be implemented
-         * scanning of whole bandwidth. Incoming results are processed by
-         * IEEE 802.11 framework itself.
-         * 
-         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
-         * 
-         * @return EOK if succeed, negative error code otherwise.
+        /** Scan neighborhood for networks.
+         *
+         * There should be implemented scanning of whole bandwidth.
+         * Incoming results are processed by IEEE 802.11 framework itself.
+         *
+         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
+         *
+         * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*scan)(struct ieee80211_dev *);
         
-        /**
-         * Handler for TX frames to be send from device. This should be
-         * called for every frame that has to be send from IEEE 802.11 device.
-         * 
-         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
-         * @param buffer Buffer with data to be send.
-         * @param buffer_size Size of buffer.
-         * 
-         * @return EOK if succeed, negative error code otherwise.
+        /** Handler for TX frames to be send from device.
+         *
+         * This should be called for every frame that has to be send
+         * from IEEE 802.11 device.
+         *
+         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
+         * @param buffer        Buffer with data to be send.
+         * @param buffer_size   Size of buffer.
+         *
+         * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*tx_handler)(struct ieee80211_dev *, void *, size_t);
         
-        /**
-         * Set device operating frequency to given value.
-         * 
-         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
-         * @param freq New device operating frequency.
-         * 
-         * @return EOK if succeed, negative error code otherwise.
+        /** Set device operating frequency to given value.
+         *
+         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
+         * @param freq          New device operating frequency.
+         *
+         * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*set_freq)(struct ieee80211_dev *, uint16_t);
         
-        /**
-         * Callback to inform device about BSSID change.
-         * 
-         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
-         * @param connected True if connected to new BSSID, otherwise false.
-         * 
-         * @return EOK if succeed, negative error code otherwise.
+        /** Callback to inform device about BSSID change.
+         *
+         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
+         * @param connected     True if connected to new BSSID, otherwise false.
+         *
+         * @return EOK if succeed, negative error code otherwise.
+         *
          */
         int (*bssid_change)(struct ieee80211_dev *, bool);
         
-        /**
-         * Callback to setup encryption key in IEEE 802.11 device.
-         * 
-         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
-         * @param key_conf Key config structure.
-         * @param insert True to insert this key to device, false to remove it.
-         * 
-         * @return EOK if succeed, negative error code otherwise.
-         */
-        int (*key_config)(struct ieee80211_dev *, 
-                ieee80211_key_config_t *key_conf, bool);
+        /** Callback to setup encryption key in IEEE 802.11 device.
+         *
+         * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
+         * @param key_conf      Key config structure.
+         * @param insert        True to insert this key to device,
+         *                      false to remove it.
+         *
+         * @return EOK if succeed, negative error code otherwise.
+         *
+         */
+        int (*key_config)(struct ieee80211_dev *,
+            ieee80211_key_config_t *key_conf, bool);
 } ieee80211_ops_t;
 
 /* Initialization functions. */
 extern ieee80211_dev_t *ieee80211_device_create(void);
-extern int ieee80211_device_init(ieee80211_dev_t *ieee80211_dev, 
-        ddf_dev_t *ddf_dev);
-extern int ieee80211_init(ieee80211_dev_t *ieee80211_dev, 
-        ieee80211_ops_t *ieee80211_ops, ieee80211_iface_t *ieee80211_iface,
-        nic_iface_t *ieee80211_nic_iface, ddf_dev_ops_t *ieee80211_dev_ops);
+extern int ieee80211_device_init(ieee80211_dev_t *, ddf_dev_t *);
+extern int ieee80211_init(ieee80211_dev_t *, ieee80211_ops_t *,
+    ieee80211_iface_t *, nic_iface_t *, ddf_dev_ops_t *);
 
 /* Getters & setters, queries & reports. */
-extern void *ieee80211_get_specific(ieee80211_dev_t *ieee80211_dev);
-extern void ieee80211_set_specific(ieee80211_dev_t *ieee80211_dev, 
-        void *specific);
-extern ddf_dev_t *ieee80211_get_ddf_dev(ieee80211_dev_t* ieee80211_dev);
-extern ieee80211_operating_mode_t 
-        ieee80211_query_current_op_mode(ieee80211_dev_t *ieee80211_dev);
-extern uint16_t ieee80211_query_current_freq(ieee80211_dev_t *ieee80211_dev);
-extern void ieee80211_query_bssid(ieee80211_dev_t* ieee80211_dev, 
-        nic_address_t *bssid);
-extern bool ieee80211_is_connected(ieee80211_dev_t* ieee80211_dev);
-extern void ieee80211_report_current_op_mode(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_operating_mode_t op_mode);
-extern void ieee80211_report_current_freq(ieee80211_dev_t *ieee80211_dev,
-        uint16_t freq);
-extern uint16_t ieee80211_get_aid(ieee80211_dev_t* ieee80211_dev);
-extern int ieee80211_get_pairwise_security(ieee80211_dev_t* ieee80211_dev);
-extern bool ieee80211_is_ready(ieee80211_dev_t* ieee80211_dev);
-extern void ieee80211_set_ready(ieee80211_dev_t* ieee80211_dev, bool ready);
-extern bool ieee80211_query_using_key(ieee80211_dev_t* ieee80211_dev);
-extern void ieee80211_setup_key_confirm(ieee80211_dev_t* ieee80211_dev, 
-        bool using_key);
-
-extern bool ieee80211_is_data_frame(uint16_t frame_ctrl);
-extern bool ieee80211_is_mgmt_frame(uint16_t frame_ctrl);
-extern bool ieee80211_is_beacon_frame(uint16_t frame_ctrl);
-extern bool ieee80211_is_probe_response_frame(uint16_t frame_ctrl);
-extern bool ieee80211_is_auth_frame(uint16_t frame_ctrl);
-extern bool ieee80211_is_assoc_response_frame(uint16_t frame_ctrl);
+extern void *ieee80211_get_specific(ieee80211_dev_t *);
+extern void ieee80211_set_specific(ieee80211_dev_t *, void *);
+extern ddf_dev_t *ieee80211_get_ddf_dev(ieee80211_dev_t *);
+extern ieee80211_operating_mode_t
+    ieee80211_query_current_op_mode(ieee80211_dev_t *);
+extern uint16_t ieee80211_query_current_freq(ieee80211_dev_t *);
+extern void ieee80211_query_bssid(ieee80211_dev_t *, nic_address_t *);
+extern bool ieee80211_is_connected(ieee80211_dev_t *);
+extern void ieee80211_report_current_op_mode(ieee80211_dev_t *,
+    ieee80211_operating_mode_t);
+extern void ieee80211_report_current_freq(ieee80211_dev_t *, uint16_t);
+extern uint16_t ieee80211_get_aid(ieee80211_dev_t *);
+extern int ieee80211_get_pairwise_security(ieee80211_dev_t *);
+extern bool ieee80211_is_ready(ieee80211_dev_t *);
+extern void ieee80211_set_ready(ieee80211_dev_t *, bool);
+extern bool ieee80211_query_using_key(ieee80211_dev_t *);
+extern void ieee80211_setup_key_confirm(ieee80211_dev_t *, bool);
+
+extern bool ieee80211_is_data_frame(uint16_t);
+extern bool ieee80211_is_mgmt_frame(uint16_t);
+extern bool ieee80211_is_beacon_frame(uint16_t);
+extern bool ieee80211_is_probe_response_frame(uint16_t);
+extern bool ieee80211_is_auth_frame(uint16_t);
+extern bool ieee80211_is_assoc_response_frame(uint16_t);
 
 /* Worker functions. */
-extern int ieee80211_rx_handler(ieee80211_dev_t *ieee80211_dev, void *buffer,
-        size_t buffer_size);
-
-#endif // LIB_IEEE80211_H
+extern int ieee80211_rx_handler(ieee80211_dev_t *, void *, size_t);
+
+#endif
 
 /** @}

uspace/lib/ieee80211/include/ieee80211_iface_impl.h

@@ -27 +27 @@
  */
 
-/** 
+/**
  * @addtogroup libieee80211
  * @{
@@ -33 +33 @@
 
 /** @file ieee80211_iface_impl.h
- * 
+ *
  * IEEE 802.11 default interface functions definition.
  */
 
 #ifndef LIB_IEEE80211_IFACE_IMPL_H
-#define LIB_IEEE80211_IFACE_IMPL_H
+#define LIB_IEEE80211_IFACE_IMPL_H
 
 #include <ddf/driver.h>
-
 #include "ieee80211.h"
 
-extern int ieee80211_get_scan_results_impl(ddf_fun_t *fun, 
-        ieee80211_scan_results_t *results, bool now);
-extern int ieee80211_connect_impl(ddf_fun_t *fun, char *ssid_start, 
-        char *password);
-extern int ieee80211_disconnect_impl(ddf_fun_t *fun);
+extern int ieee80211_get_scan_results_impl(ddf_fun_t *,
+    ieee80211_scan_results_t *, bool);
+extern int ieee80211_connect_impl(ddf_fun_t *, char *, char *);
+extern int ieee80211_disconnect_impl(ddf_fun_t *);
 
-#endif  /* LIB_IEEE80211_IFACE_IMPL_H */
+#endif  /* LIB_IEEE80211_IFACE_IMPL_H */
 
 /** @}

uspace/lib/ieee80211/include/ieee80211_impl.h

@@ -27 +27 @@
  */
 
-/** 
+/**
  * @addtogroup libieee80211
  * @{
@@ -33 +33 @@
 
 /** @file ieee80211_impl.h
- * 
+ *
  * IEEE 802.11 default device functions definition.
  */
 
 #ifndef LIB_IEEE80211_IMPL_H
-#define LIB_IEEE80211_IMPL_H
+#define LIB_IEEE80211_IMPL_H
 
 #include "ieee80211_private.h"
 
-extern int ieee80211_start_impl(ieee80211_dev_t *ieee80211_dev);
-extern int ieee80211_tx_handler_impl(ieee80211_dev_t *ieee80211_dev, 
-        void *buffer, size_t buffer_size);
-extern int ieee80211_set_freq_impl(ieee80211_dev_t *ieee80211_dev, 
-        uint16_t freq);
-extern int ieee80211_bssid_change_impl(ieee80211_dev_t *ieee80211_dev,
-        bool connected);
-extern int ieee80211_key_config_impl(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_key_config_t *key_conf, bool insert);
-extern int ieee80211_scan_impl(ieee80211_dev_t *ieee80211_dev);
-extern int ieee80211_prf(uint8_t *key, uint8_t *data, uint8_t *hash, 
-        size_t output_size);
-extern int ieee80211_rc4_key_unwrap(uint8_t *key, uint8_t *data, 
-        size_t data_size, uint8_t *output);
-extern int ieee80211_aes_key_unwrap(uint8_t *kek, uint8_t *data, 
-        size_t data_size, uint8_t *output);
-extern int ieee80211_michael_mic(uint8_t *key, uint8_t *data, size_t data_size, 
-        uint8_t *mic);
-extern uint16_t uint16le_from_seq(void *seq);
-extern uint32_t uint32le_from_seq(void *seq);
-extern uint16_t uint16be_from_seq(void *seq);
-extern uint32_t uint32be_from_seq(void *seq);
-extern int rnd_sequence(uint8_t *sequence, size_t length);
-extern uint8_t *min_sequence(uint8_t *seq1, uint8_t *seq2, size_t size);
-extern uint8_t *max_sequence(uint8_t *seq1, uint8_t *seq2, size_t size);
+extern int ieee80211_start_impl(ieee80211_dev_t *);
+extern int ieee80211_tx_handler_impl(ieee80211_dev_t *, void *, size_t);
+extern int ieee80211_set_freq_impl(ieee80211_dev_t *, uint16_t);
+extern int ieee80211_bssid_change_impl(ieee80211_dev_t *, bool);
+extern int ieee80211_key_config_impl(ieee80211_dev_t *,
+    ieee80211_key_config_t *, bool);
+extern int ieee80211_scan_impl(ieee80211_dev_t *);
+extern int ieee80211_prf(uint8_t *, uint8_t *, uint8_t *, size_t);
+extern int ieee80211_rc4_key_unwrap(uint8_t *, uint8_t *, size_t, uint8_t *);
+extern int ieee80211_aes_key_unwrap(uint8_t *, uint8_t *, size_t, uint8_t *);
+extern int ieee80211_michael_mic(uint8_t *, uint8_t *, size_t, uint8_t *);
+extern uint16_t uint16le_from_seq(void *);
+extern uint32_t uint32le_from_seq(void *);
+extern uint16_t uint16be_from_seq(void *);
+extern uint32_t uint32be_from_seq(void *);
+extern int rnd_sequence(uint8_t *, size_t);
+extern uint8_t *min_sequence(uint8_t *, uint8_t *, size_t);
+extern uint8_t *max_sequence(uint8_t *, uint8_t *, size_t);
 
-#endif  /* LIB_IEEE80211_IMPL_H */
+#endif  /* LIB_IEEE80211_IMPL_H */
 
 /** @}

uspace/lib/ieee80211/include/ieee80211_private.h

@@ -27 +27 @@
  */
 
-/** 
+/**
  * @addtogroup libieee80211
  * @{
@@ -33 +33 @@
 
 /** @file ieee80211.h
- * 
+ *
  * Internal IEEE 802.11 header that should not be included.
  */
 
-#ifndef LIBNET_IEEE80211_PRIVATE_H
-#define LIBNET_IEEE80211_PRIVATE_H
+#ifndef LIB_IEEE80211_PRIVATE_H
+#define LIB_IEEE80211_PRIVATE_H
 
 #include <fibril_synch.h>
@@ -44 +44 @@
 #include <ddf/driver.h>
 #include <sys/types.h>
-
 #include <ieee80211/ieee80211.h>
 #include "ieee80211.h"
 
-/* Timeout in us for waiting to authentication/association response. */
-#define AUTH_TIMEOUT 200000
-
-/* Timeout in us for waiting to finish 4-way handshake process. */
-#define HANDSHAKE_TIMEOUT 5000000
-
-/* Scanning period. */
-#define SCAN_PERIOD_USEC 35000000
-
-/* Time to wait for beacons on channel. */
-#define SCAN_CHANNEL_WAIT_USEC 200000
-
-/* Max time to keep scan result. */
-#define MAX_KEEP_SCAN_SPAN_SEC 120
-
-/* Security bit in capability info field. */
-#define CAP_SECURITY 0x10
-
-/* Protocol type used in EAPOL frames. */
-#define ETH_TYPE_PAE 0x888E
-
-/* WPA OUI used in vendor specific IE. */
-#define WPA_OUI 0x0050F201
-
-/* GTK OUI used in vendor specific IE. */
-#define GTK_OUI 0x000FAC01
-
-/* Max PTK key length. */
-#define MAX_PTK_LENGTH 64
-
-/* Max GTK key length. */
-#define MAX_GTK_LENGTH 64
-
-/* KEK offset inside PTK. */
-#define KEK_OFFSET 16
-
-/* TK offset inside PTK. */
-#define TK_OFFSET 32
-
-/* Length of Michael MIC code used in TKIP security suite. */
-#define MIC_LENGTH 8
-
-/* 
- * Length of data to be encrypted by PRF function:
- * NONCE + SNONCE (2 * 32) + DEST_MAC + SOURCE_MAC (2 * ETH_ADDR) 
+/** Timeout in us for waiting to authentication/association response. */
+#define AUTH_TIMEOUT  200000
+
+/** Timeout in us for waiting to finish 4-way handshake process. */
+#define HANDSHAKE_TIMEOUT  5000000
+
+/** Scanning period. */
+#define SCAN_PERIOD_USEC  35000000
+
+/** Time to wait for beacons on channel. */
+#define SCAN_CHANNEL_WAIT_USEC  200000
+
+/** Max time to keep scan result. */
+#define MAX_KEEP_SCAN_SPAN_SEC  120
+
+/** Security bit in capability info field. */
+#define CAP_SECURITY  0x10
+
+/** Protocol type used in EAPOL frames. */
+#define ETH_TYPE_PAE  0x888e
+
+/** WPA OUI used in vendor specific IE. */
+#define WPA_OUI  0x0050f201
+
+/** GTK OUI used in vendor specific IE. */
+#define GTK_OUI  0x000fac01
+
+/** Max PTK key length. */
+#define MAX_PTK_LENGTH  64
+
+/** Max GTK key length. */
+#define MAX_GTK_LENGTH  64
+
+/** KEK offset inside PTK. */
+#define KEK_OFFSET  16
+
+/** TK offset inside PTK. */
+#define TK_OFFSET  32
+
+/** Length of Michael MIC code used in TKIP security suite. */
+#define MIC_LENGTH  8
+
+/** Length of data to be encrypted by PRF function.
+ *
+ * NONCE + SNONCE (2 * 32) + DEST_MAC + SOURCE_MAC (2 * ETH_ADDR)
+ *
  */
-#define PRF_CRYPT_DATA_LENGTH 2*32 + 2*ETH_ADDR
-
-/* Special room in header reserved for encryption. */
+#define PRF_CRYPT_DATA_LENGTH  (2 * 32 + 2 * ETH_ADDR)
+
+/** Special room in header reserved for encryption. */
 typedef enum {
         IEEE80211_TKIP_HEADER_LENGTH = 8,
@@ -184 +184 @@
 /** IEEE 802.11 information element types. */
 typedef enum {
-        IEEE80211_SSID_IE = 0,          /**< Target SSID. */
-        IEEE80211_RATES_IE = 1,         /**< Supported data rates. */
-        IEEE80211_CHANNEL_IE = 3,       /**< Current channel number. */
-        IEEE80211_CHALLENGE_IE = 16,    /**< Challenge text. */
-        IEEE80211_RSN_IE = 48,          /**< RSN. */
-        IEEE80211_EXT_RATES_IE = 50,    /**< Extended data rates. */
-        IEEE80211_VENDOR_IE = 221       /**< Vendor specific IE. */
+        IEEE80211_SSID_IE = 0,        /**< Target SSID. */
+        IEEE80211_RATES_IE = 1,       /**< Supported data rates. */
+        IEEE80211_CHANNEL_IE = 3,     /**< Current channel number. */
+        IEEE80211_CHALLENGE_IE = 16,  /**< Challenge text. */
+        IEEE80211_RSN_IE = 48,        /**< RSN. */
+        IEEE80211_EXT_RATES_IE = 50,  /**< Extended data rates. */
+        IEEE80211_VENDOR_IE = 221     /**< Vendor specific IE. */
 } ieee80211_ie_type_t;
 
@@ -249 +249 @@
         ieee80211_bssid_info_t bssid_info;
         
-        /** 
+        /**
          * Flag indicating that data traffic is encrypted by HW key
-         * that is set up in device. 
+         * that is set up in device.
          */
         bool using_hw_key;
@@ -290 +290 @@
         uint8_t dest_addr[ETH_ADDR];
         uint8_t src_addr[ETH_ADDR];
-        uint16_t proto;                 /**< Big Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        eth_header_t;
+        uint16_t proto;  /**< Big Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    eth_header_t;
 
 /** IEEE 802.11 management header structure. */
 typedef struct {
-        uint16_t frame_ctrl;            /**< Little Endian value! */
-        uint16_t duration_id;           /**< Little Endian value! */
+        uint16_t frame_ctrl;          /**< Little Endian value! */
+        uint16_t duration_id;         /**< Little Endian value! */
         uint8_t dest_addr[ETH_ADDR];
         uint8_t src_addr[ETH_ADDR];
         uint8_t bssid[ETH_ADDR];
-        uint16_t seq_ctrl;              /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_mgmt_header_t;
+        uint16_t seq_ctrl;            /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_mgmt_header_t;
 
 /** IEEE 802.11 data header structure. */
 typedef struct {
-        uint16_t frame_ctrl;            /**< Little Endian value! */
-        uint16_t duration_id;           /**< Little Endian value! */
+        uint16_t frame_ctrl;         /**< Little Endian value! */
+        uint16_t duration_id;        /**< Little Endian value! */
         uint8_t address1[ETH_ADDR];
         uint8_t address2[ETH_ADDR];
         uint8_t address3[ETH_ADDR];
-        uint16_t seq_ctrl;              /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_data_header_t;
+        uint16_t seq_ctrl;           /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_data_header_t;
 
 /** IEEE 802.11 information element header. */
@@ -320 +320 @@
         uint8_t element_id;
         uint8_t length;
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_ie_header_t;
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_ie_header_t;
 
 /** IEEE 802.11 authentication frame body. */
 typedef struct {
-        uint16_t auth_alg;              /**< Little Endian value! */
-        uint16_t auth_trans_no;         /**< Little Endian value! */
-        uint16_t status;                /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_auth_body_t;
+        uint16_t auth_alg;       /**< Little Endian value! */
+        uint16_t auth_trans_no;  /**< Little Endian value! */
+        uint16_t status;         /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_auth_body_t;
 
 /** IEEE 802.11 deauthentication frame body. */
 typedef struct {
-        uint16_t reason;                /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_deauth_body_t;
+        uint16_t reason;    /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_deauth_body_t;
 
 /** IEEE 802.11 association request frame body. */
 typedef struct {
-        uint16_t capability;            /**< Little Endian value! */
-        uint16_t listen_interval;       /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_assoc_req_body_t;
+        uint16_t capability;       /**< Little Endian value! */
+        uint16_t listen_interval;  /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_assoc_req_body_t;
 
 /** IEEE 802.11 association response frame body. */
 typedef struct {
-        uint16_t capability;            /**< Little Endian value! */
-        uint16_t status;                /**< Little Endian value! */
-        uint16_t aid;                   /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_assoc_resp_body_t;
+        uint16_t capability;  /**< Little Endian value! */
+        uint16_t status;      /**< Little Endian value! */
+        uint16_t aid;         /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_assoc_resp_body_t;
 
 /** IEEE 802.11 beacon frame body start. */
 typedef struct {
         uint8_t timestamp[8];
-        uint16_t beacon_interval;       /**< Little Endian value! */
-        uint16_t capability;            /**< Little Endian value! */
-} __attribute__((packed)) __attribute__ ((aligned(2))) 
-        ieee80211_beacon_start_t;
+        uint16_t beacon_interval;  /**< Little Endian value! */
+        uint16_t capability;       /**< Little Endian value! */
+} __attribute__((packed)) __attribute__((aligned(2)))
+    ieee80211_beacon_start_t;
 
 /** IEEE 802.11i EAPOL-Key frame format. */
@@ -364 +364 @@
         uint8_t proto_version;
         uint8_t packet_type;
-        uint16_t body_length;           /**< Big Endian value! */
+        uint16_t body_length;      /**< Big Endian value! */
         uint8_t descriptor_type;
-        uint16_t key_info;              /**< Big Endian value! */
-        uint16_t key_length;            /**< Big Endian value! */
+        uint16_t key_info;         /**< Big Endian value! */
+        uint16_t key_length;       /**< Big Endian value! */
         uint8_t key_replay_counter[8];
         uint8_t key_nonce[32];
@@ -374 +374 @@
         uint8_t reserved[8];
         uint8_t key_mic[16];
-        uint16_t key_data_length;       /**< Big Endian value! */
+        uint16_t key_data_length;  /**< Big Endian value! */
 } __attribute__((packed)) ieee80211_eapol_key_frame_t;
 
 #define ieee80211_scan_result_list_foreach(results, iter) \
-    list_foreach((results).list, link, ieee80211_scan_result_link_t, (iter))
-
-static inline void ieee80211_scan_result_list_init(
-        ieee80211_scan_result_list_t *results) 
+        list_foreach((results).list, link, ieee80211_scan_result_link_t, (iter))
+
+static inline void
+    ieee80211_scan_result_list_init(ieee80211_scan_result_list_t *results)
 {
         list_initialize(&results->list);
@@ -387 +387 @@
 }
 
-static inline void ieee80211_scan_result_list_remove(
-        ieee80211_scan_result_list_t *results,
-        ieee80211_scan_result_link_t *result)
+static inline void
+    ieee80211_scan_result_list_remove(ieee80211_scan_result_list_t *results,
+    ieee80211_scan_result_link_t *result)
 {
         list_remove(&result->link);
@@ -395 +395 @@
 }
 
-static inline void ieee80211_scan_result_list_append(
-        ieee80211_scan_result_list_t *results,
-        ieee80211_scan_result_link_t *result)
+static inline void
+    ieee80211_scan_result_list_append(ieee80211_scan_result_list_t *results,
+    ieee80211_scan_result_link_t *result)
 {
         list_append(&result->link, &results->list);
@@ -403 +403 @@
 }
 
-extern bool ieee80211_is_fromds_frame(uint16_t frame_ctrl);
-extern bool ieee80211_is_tods_frame(uint16_t frame_ctrl);
-extern void ieee80211_set_connect_request(ieee80211_dev_t *ieee80211_dev);
-extern bool ieee80211_pending_connect_request(ieee80211_dev_t *ieee80211_dev);
-extern ieee80211_auth_phase_t ieee80211_get_auth_phase(ieee80211_dev_t 
-        *ieee80211_dev);
-extern void ieee80211_set_auth_phase(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_auth_phase_t auth_phase);
-extern int ieee80211_probe_request(ieee80211_dev_t *ieee80211_dev, 
-        char *ssid);
-extern int ieee80211_authenticate(ieee80211_dev_t *ieee80211_dev);
-extern int ieee80211_associate(ieee80211_dev_t *ieee80211_dev, 
-        char *password);
-extern int ieee80211_deauthenticate(ieee80211_dev_t *ieee80211_dev);
-
-#endif /* LIBN_IEEE80211_H */
+extern bool ieee80211_is_fromds_frame(uint16_t);
+extern bool ieee80211_is_tods_frame(uint16_t);
+extern void ieee80211_set_connect_request(ieee80211_dev_t *);
+extern bool ieee80211_pending_connect_request(ieee80211_dev_t *);
+extern ieee80211_auth_phase_t ieee80211_get_auth_phase(ieee80211_dev_t *);
+extern void ieee80211_set_auth_phase(ieee80211_dev_t *, ieee80211_auth_phase_t);
+extern int ieee80211_probe_request(ieee80211_dev_t *, char *);
+extern int ieee80211_authenticate(ieee80211_dev_t *);
+extern int ieee80211_associate(ieee80211_dev_t *, char *);
+extern int ieee80211_deauthenticate(ieee80211_dev_t *);
+
+#endif
 
 /** @}

uspace/lib/ieee80211/src/ieee80211.c

@@ -32 +32 @@
 
 /** @file ieee80211.c
- * 
+ *
  * IEEE 802.11 interface implementation.
  */
@@ -41 +41 @@
 #include <macros.h>
 #include <errno.h>
-
 #include <ieee80211.h>
 #include <ieee80211_impl.h>
@@ -48 +47 @@
 #include <ops/ieee80211.h>
 
-#define IEEE80211_DATA_RATES_SIZE 8
-#define IEEE80211_EXT_DATA_RATES_SIZE 4
+#define IEEE80211_DATA_RATES_SIZE      8
+#define IEEE80211_EXT_DATA_RATES_SIZE  4
 
 #define ATOMIC_GET(state)
 
 /** Frame encapsulation used in IEEE 802.11. */
-static const uint8_t rfc1042_header[] = { 
-        0xAA, 0xAA, 0x03, 0x00, 0x00, 0x00 
+static const uint8_t rfc1042_header[] = {
+        0xaa, 0xaa, 0x03, 0x00, 0x00, 0x00 
 };
 
 /** Broadcast MAC address. */
 static const uint8_t ieee80211_broadcast_mac_addr[] = {
-        0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF
+        0xff, 0xff, 0xff, 0xff, 0xff, 0xff
 };
 
-/**
- * Check data frame.
- * 
+/** Check data frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is data frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_data_frame(uint16_t frame_ctrl)
@@ -74 +73 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_TYPE)
-                == IEEE80211_DATA_FRAME;
-}
-
-/**
- * Check management frame.
- * 
+        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_TYPE) ==
+            IEEE80211_DATA_FRAME;
+}
+
+/** Check management frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is management frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_mgmt_frame(uint16_t frame_ctrl)
@@ -89 +88 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_TYPE)
-                == IEEE80211_MGMT_FRAME;
-}
-
-/**
- * Check management beacon frame.
- * 
+        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_TYPE) ==
+            IEEE80211_MGMT_FRAME;
+}
+
+/** Check management beacon frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is beacon frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_beacon_frame(uint16_t frame_ctrl)
@@ -104 +103 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE)
-                == IEEE80211_MGMT_BEACON_FRAME;
-}
-
-/**
- * Check management probe response frame.
- * 
+        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE) ==
+            IEEE80211_MGMT_BEACON_FRAME;
+}
+
+/** Check management probe response frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is probe resp frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_probe_response_frame(uint16_t frame_ctrl)
@@ -119 +118 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE)
-                == IEEE80211_MGMT_PROBE_RESP_FRAME;
-}
-
-/**
- * Check management authentication frame.
- * 
+        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE) ==
+            IEEE80211_MGMT_PROBE_RESP_FRAME;
+}
+
+/** Check management authentication frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is auth frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_auth_frame(uint16_t frame_ctrl)
@@ -134 +133 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE)
-                == IEEE80211_MGMT_AUTH_FRAME;
-}
-
-/**
- * Check management association response frame.
- * 
+        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE) ==
+            IEEE80211_MGMT_AUTH_FRAME;
+}
+
+/** Check management association response frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is assoc resp frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_assoc_response_frame(uint16_t frame_ctrl)
@@ -149 +148 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE)
-                == IEEE80211_MGMT_ASSOC_RESP_FRAME;
-}
-
-/**
- * Check data frame "to distribution system" direction.
- * 
+        return (frame_ctrl & IEEE80211_FRAME_CTRL_FRAME_SUBTYPE) ==
+            IEEE80211_MGMT_ASSOC_RESP_FRAME;
+}
+
+/** Check data frame "to distribution system" direction.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is TODS frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_tods_frame(uint16_t frame_ctrl)
@@ -167 +166 @@
 }
 
-/**
- * Check data frame "from distribution system" direction.
- * 
+/** Check data frame "from distribution system" direction.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it is FROMDS frame, otherwise false.
+ *
  */
 inline bool ieee80211_is_fromds_frame(uint16_t frame_ctrl)
@@ -181 +180 @@
 }
 
-/**
- * Check if it is data frame containing payload data.
- * 
+/** Check if it is data frame containing payload data.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if it has payload data, otherwise false.
+ *
  */
 static inline bool ieee80211_has_data_frame(uint16_t frame_ctrl)
@@ -192 +191 @@
         frame_ctrl = uint16_t_le2host(frame_ctrl);
         
-        return (frame_ctrl & (IEEE80211_FRAME_CTRL_FRAME_TYPE | 0x40))
-                == IEEE80211_DATA_FRAME;
-}
-
-/**
- * Check if it is encrypted frame.
- * 
+        return (frame_ctrl & (IEEE80211_FRAME_CTRL_FRAME_TYPE | 0x40)) ==
+            IEEE80211_DATA_FRAME;
+}
+
+/** Check if it is encrypted frame.
+ *
  * @param frame_ctrl Frame control field in little endian (!).
- * 
+ *
  * @return True if the frame is encrypted, otherwise false.
+ *
  */
 static inline bool ieee80211_is_encrypted_frame(uint16_t frame_ctrl)
@@ -210 +209 @@
 }
 
-/**
- * Check if PAE packet is EAPOL-Key frame.
- * 
+/** Check if PAE packet is EAPOL-Key frame.
+ *
  * @param key_frame Pointer to start of EAPOL frame.
- * 
+ *
  * @return True if it is EAPOL-Key frame, otherwise false.
- */
-static inline bool ieee80211_is_eapol_key_frame(ieee80211_eapol_key_frame_t 
-        *key_frame)
+ *
+ */
+static inline bool
+    ieee80211_is_eapol_key_frame(ieee80211_eapol_key_frame_t *key_frame)
 {
         return (key_frame->packet_type == IEEE80211_EAPOL_KEY);
 }
 
-
-/**
- * Generate packet sequence number.
- * 
+/** Generate packet sequence number.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return True if it has payload data, otherwise false.
+ *
  */
 static uint16_t ieee80211_get_sequence_number(ieee80211_dev_t *ieee80211_dev)
@@ -235 +233 @@
         uint16_t ret_val = ieee80211_dev->sequence_number;
         ieee80211_dev->sequence_number += (1 << 4);
+        
         return ret_val;
 }
 
-/**
- * Get driver-specific structure for IEEE 802.11 device.
- * 
+/** Get driver-specific structure for IEEE 802.11 device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return Driver-specific structure.
- */
-void *ieee80211_get_specific(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+void *ieee80211_get_specific(ieee80211_dev_t *ieee80211_dev)
 {
         return ieee80211_dev->specific;
 }
 
-/**
- * Set driver-specific structure for IEEE 802.11 device.
- * 
+/** Set driver-specific structure for IEEE 802.11 device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * @param specific Driver-specific structure.
- */
-void ieee80211_set_specific(ieee80211_dev_t* ieee80211_dev,
-        void *specific)
+ * @param specific      Driver-specific structure.
+ *
+ */
+void ieee80211_set_specific(ieee80211_dev_t *ieee80211_dev,
+    void *specific)
 {
         ieee80211_dev->specific = specific;
 }
 
-/**
- * Get related DDF device.
- * 
+/** Get related DDF device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return DDF device.
- */
-ddf_dev_t *ieee80211_get_ddf_dev(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+ddf_dev_t *ieee80211_get_ddf_dev(ieee80211_dev_t *ieee80211_dev)
 {
         return ieee80211_dev->ddf_dev;
 }
 
-/**
- * Query current operating mode of IEEE 802.11 device.
- * 
+/** Query current operating mode of IEEE 802.11 device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return Current IEEE 802.11 operating mode.
- */
-ieee80211_operating_mode_t ieee80211_query_current_op_mode(ieee80211_dev_t* 
-        ieee80211_dev)
+ *
+ */
+ieee80211_operating_mode_t
+    ieee80211_query_current_op_mode(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -291 +290 @@
 }
 
-/**
- * Query current frequency of IEEE 802.11 device.
- * 
+/** Query current frequency of IEEE 802.11 device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return Current device operating frequency.
- */
-uint16_t ieee80211_query_current_freq(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+uint16_t ieee80211_query_current_freq(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -307 +306 @@
 }
 
-/**
- * Query BSSID the device is connected to.
- * 
+/** Query BSSID the device is connected to.
+ *
  * Note: Expecting locked results_mutex.
- * 
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * @param bssid Pointer to structure where should be stored BSSID.
- */
-void ieee80211_query_bssid(ieee80211_dev_t* ieee80211_dev, 
-        nic_address_t *bssid)
+ * @param bssid         Pointer to structure where should be stored BSSID.
+ *
+ */
+void ieee80211_query_bssid(ieee80211_dev_t *ieee80211_dev,
+    nic_address_t *bssid)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
         
-        if(bssid) {
-                ieee80211_scan_result_link_t *res_link = 
-                        ieee80211_dev->bssid_info.res_link;
-                
-                if(res_link) {
-                        memcpy(bssid, &res_link->scan_result.bssid, 
-                                sizeof(nic_address_t));
+        if (bssid) {
+                ieee80211_scan_result_link_t *res_link =
+                    ieee80211_dev->bssid_info.res_link;
+                
+                if (res_link) {
+                        memcpy(bssid, &res_link->scan_result.bssid,
+                            sizeof(nic_address_t));
                 } else {
                         nic_address_t broadcast_addr;
                         memcpy(broadcast_addr.address,
-                                ieee80211_broadcast_mac_addr,
-                                ETH_ADDR);
+                            ieee80211_broadcast_mac_addr, ETH_ADDR);
                         memcpy(bssid, &broadcast_addr, sizeof(nic_address_t));
                 }
@@ -339 +337 @@
 }
 
-/**
- * Get AID of network we are connected to.
- * 
+/** Get AID of network we are connected to.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return AID.
- */
-uint16_t ieee80211_get_aid(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+uint16_t ieee80211_get_aid(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -355 +353 @@
 }
 
-/**
- * Get pairwise security suite used for HW encryption. 
- * 
+/** Get pairwise security suite used for HW encryption. 
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return Security suite indicator.
- */
-int ieee80211_get_pairwise_security(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+int ieee80211_get_pairwise_security(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
-        ieee80211_scan_result_link_t *auth_link = 
-                ieee80211_dev->bssid_info.res_link;
+        ieee80211_scan_result_link_t *auth_link =
+            ieee80211_dev->bssid_info.res_link;
         int suite = auth_link->scan_result.security.pair_alg;
         fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
@@ -373 +371 @@
 }
 
-/**
- * Check if IEEE 802.11 device is connected to network.
- * 
+/** Check if IEEE 802.11 device is connected to network.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return True if device is connected to network, otherwise false.
- */
-bool ieee80211_is_connected(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+bool ieee80211_is_connected(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
-        bool conn_state = 
-                ieee80211_dev->current_auth_phase == IEEE80211_AUTH_CONNECTED;
+        bool conn_state =
+            ieee80211_dev->current_auth_phase == IEEE80211_AUTH_CONNECTED;
         fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
+        
         return conn_state;
 }
 
 void ieee80211_set_auth_phase(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_auth_phase_t auth_phase)
+    ieee80211_auth_phase_t auth_phase)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -397 +396 @@
 }
 
-ieee80211_auth_phase_t ieee80211_get_auth_phase(ieee80211_dev_t 
-        *ieee80211_dev)
+ieee80211_auth_phase_t ieee80211_get_auth_phase(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
         ieee80211_auth_phase_t conn_state = ieee80211_dev->current_auth_phase;
         fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
+        
         return conn_state;
 }
@@ -419 +418 @@
         ieee80211_dev->pending_conn_req = false;
         fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
+        
         return conn_request;
 }
 
-/**
- * Report current operating mode for IEEE 802.11 device.
- * 
+/** Report current operating mode for IEEE 802.11 device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * @param op_mode Current IEEE 802.11 operating mode.
- */
-void ieee80211_report_current_op_mode(ieee80211_dev_t* ieee80211_dev,
-        ieee80211_operating_mode_t op_mode)
+ * @param op_mode       Current IEEE 802.11 operating mode.
+ *
+ */
+void ieee80211_report_current_op_mode(ieee80211_dev_t *ieee80211_dev,
+    ieee80211_operating_mode_t op_mode)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -436 +436 @@
 }
 
-/**
- * Report current frequency for IEEE 802.11 device.
- * 
+/** Report current frequency for IEEE 802.11 device.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * @param freq Current device operating frequency.
- */
-void ieee80211_report_current_freq(ieee80211_dev_t* ieee80211_dev,
-        uint16_t freq)
+ * @param freq          Current device operating frequency.
+ *
+ */
+void ieee80211_report_current_freq(ieee80211_dev_t *ieee80211_dev,
+    uint16_t freq)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -450 +450 @@
 }
 
-/**
- * Check if IEEE 802.11 device is ready (fully initialized).
- * 
+/** Check if IEEE 802.11 device is ready (fully initialized).
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * 
+ *
  * @return True if device is ready to work, otherwise false.
- */
-bool ieee80211_is_ready(ieee80211_dev_t* ieee80211_dev)
+ *
+ */
+bool ieee80211_is_ready(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -466 +466 @@
 }
 
-/**
- * Set IEEE 802.11 device to ready state.
- * 
+/** Set IEEE 802.11 device to ready state.
+ *
  * @param ieee80211_dev IEEE 802.11 device.
- * @param ready Ready state to be set.
- */
-void ieee80211_set_ready(ieee80211_dev_t* ieee80211_dev, bool ready)
+ * @param ready         Ready state to be set.
+ *
+ */
+void ieee80211_set_ready(ieee80211_dev_t *ieee80211_dev, bool ready)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -479 +479 @@
 }
 
-extern bool ieee80211_query_using_key(ieee80211_dev_t* ieee80211_dev)
+extern bool ieee80211_query_using_key(ieee80211_dev_t *ieee80211_dev)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -488 +488 @@
 }
 
-void ieee80211_setup_key_confirm(ieee80211_dev_t* ieee80211_dev, 
-        bool using_key)
+void ieee80211_setup_key_confirm(ieee80211_dev_t *ieee80211_dev,
+    bool using_key)
 {
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -502 +502 @@
         ieee80211_dev_t *ieee80211_dev = (ieee80211_dev_t *) arg;
         
-        while(true) {
+        while (true) {
                 ieee80211_dev->ops->scan(ieee80211_dev);
                 async_usleep(SCAN_PERIOD_USEC);
@@ -510 +510 @@
 }
 
-/**
- * Implementation of NIC open callback for IEEE 802.11 devices.
- * 
+/** Implementation of NIC open callback for IEEE 802.11 devices.
+ *
  * @param fun NIC function.
- * 
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 static int ieee80211_open(ddf_fun_t *fun)
@@ -522 +522 @@
         ieee80211_dev_t *ieee80211_dev = nic_get_specific(nic_data);
         
-        if(ieee80211_dev->started) {
+        if (ieee80211_dev->started)
                 return EOK;
-        } else {
-                ieee80211_dev->started = true;
-        }
+        
+        ieee80211_dev->started = true;
         
         int rc = ieee80211_dev->ops->start(ieee80211_dev);
-        if(rc != EOK)
+        if (rc != EOK)
                 return rc;
         
         /* Add scanning fibril. */
         fid_t fibril = fibril_create(ieee80211_scan, ieee80211_dev);
-        if (fibril == 0) {
+        if (fibril == 0)
                 return ENOMEM;
-        }
+        
         fibril_add_ready(fibril);
         
@@ -542 +541 @@
 }
 
-/**
- * Send frame handler.
- * 
- * @param nic Pointer to NIC device.
+/** Send frame handler.
+ *
+ * @param nic  Pointer to NIC device.
  * @param data Data buffer.
  * @param size Data buffer size.
+ *
  */
 static void ieee80211_send_frame(nic_t *nic, void *data, size_t size)
 {
-        ieee80211_dev_t *ieee80211_dev = (ieee80211_dev_t *) 
-                nic_get_specific(nic);
-        
-        ieee80211_auth_phase_t auth_phase = 
-                ieee80211_get_auth_phase(ieee80211_dev);
-        if(auth_phase != IEEE80211_AUTH_ASSOCIATED && 
-                auth_phase != IEEE80211_AUTH_CONNECTED) {
+        ieee80211_dev_t *ieee80211_dev = (ieee80211_dev_t *)
+            nic_get_specific(nic);
+        
+        ieee80211_auth_phase_t auth_phase =
+            ieee80211_get_auth_phase(ieee80211_dev);
+        if ((auth_phase != IEEE80211_AUTH_ASSOCIATED) &&
+            (auth_phase != IEEE80211_AUTH_CONNECTED))
                 return;
-        }
         
         ieee80211_scan_result_t *auth_data =
-                        &ieee80211_dev->bssid_info.res_link->scan_result;
+            &ieee80211_dev->bssid_info.res_link->scan_result;
         
         /* We drop part of IEEE 802.3 ethernet header. */
         size_t drop_bytes = sizeof(eth_header_t) - 2;
         
-        size_t complete_size = (size - drop_bytes) + 
-                sizeof(ieee80211_data_header_t) +
-                ARRAY_SIZE(rfc1042_header);
+        size_t complete_size = (size - drop_bytes) +
+            sizeof(ieee80211_data_header_t) +
+            ARRAY_SIZE(rfc1042_header);
         
         /* Init crypto data. */
@@ -579 +577 @@
         
         // TODO: Distinguish used key (pair/group) by dest address ?
-        if(ieee80211_query_using_key(ieee80211_dev)) {
+        if (ieee80211_query_using_key(ieee80211_dev)) {
                 int sec_suite = auth_data->security.pair_alg;
-                switch(sec_suite) {
-                        case IEEE80211_SECURITY_SUITE_TKIP:
-                                head_space = IEEE80211_TKIP_HEADER_LENGTH;
-                                mic_space = MIC_LENGTH;
-                                add_mic = true;
-                                break;
-                        case IEEE80211_SECURITY_SUITE_CCMP:
-                                head_space = IEEE80211_CCMP_HEADER_LENGTH;
-                                head_data[3] = 0x20;
-                                break;
-                        default:
-                                break;
+                switch (sec_suite) {
+                case IEEE80211_SECURITY_SUITE_TKIP:
+                        head_space = IEEE80211_TKIP_HEADER_LENGTH;
+                        mic_space = MIC_LENGTH;
+                        add_mic = true;
+                        break;
+                case IEEE80211_SECURITY_SUITE_CCMP:
+                        head_space = IEEE80211_CCMP_HEADER_LENGTH;
+                        head_data[3] = 0x20;
+                        break;
+                default:
+                        break;
                 }
-
+                
                 crypto = uint16_t_le2host(IEEE80211_FRAME_CTRL_PROTECTED);
         }
@@ -601 +599 @@
         
         void *complete_buffer = malloc(complete_size);
+        if (!complete_buffer)
+                return;
+        
         memset(complete_buffer, 0, complete_size);
         
-        if(head_space) {
+        if (head_space)
                 memcpy(complete_buffer + sizeof(ieee80211_data_header_t),
-                        head_data, head_space);
-        }
+                    head_data, head_space);
         
         memcpy(complete_buffer + sizeof(ieee80211_data_header_t) + head_space,
-                rfc1042_header,
-                ARRAY_SIZE(rfc1042_header));
-        
-        memcpy(complete_buffer + 
-                sizeof(ieee80211_data_header_t) +
-                ARRAY_SIZE(rfc1042_header) + head_space, 
-                data + drop_bytes, size - drop_bytes);
+            rfc1042_header, ARRAY_SIZE(rfc1042_header));
+        
+        memcpy(complete_buffer + sizeof(ieee80211_data_header_t) +
+            ARRAY_SIZE(rfc1042_header) + head_space,
+            data + drop_bytes, size - drop_bytes);
         
         ieee80211_data_header_t *data_header =
-                (ieee80211_data_header_t *) complete_buffer;
-        data_header->frame_ctrl = 
-                uint16_t_le2host(IEEE80211_DATA_FRAME) |
-                uint16_t_le2host(IEEE80211_DATA_DATA_FRAME) |
-                uint16_t_le2host(IEEE80211_FRAME_CTRL_TODS) |
-                crypto;
+            (ieee80211_data_header_t *) complete_buffer;
+        data_header->frame_ctrl =
+            uint16_t_le2host(IEEE80211_DATA_FRAME) |
+            uint16_t_le2host(IEEE80211_DATA_DATA_FRAME) |
+            uint16_t_le2host(IEEE80211_FRAME_CTRL_TODS) |
+            crypto;
         data_header->seq_ctrl = ieee80211_get_sequence_number(ieee80211_dev);
         
         /* BSSID, SA, DA. */
-        memcpy(data_header->address1, auth_data->bssid.address, ETH_ADDR);
+        memcpy(data_header->address1, auth_data->bssid.address, ETH_ADDR);
         memcpy(data_header->address2, data + ETH_ADDR, ETH_ADDR);
         memcpy(data_header->address3, data, ETH_ADDR);
         
-        if(add_mic) {
+        if (add_mic) {
                 size_t size_wo_mic = complete_size - MIC_LENGTH;
                 uint8_t *tx_mic = ieee80211_dev->bssid_info.ptk +
-                        TK_OFFSET +
-                        IEEE80211_TKIP_TX_MIC_OFFSET;
+                    TK_OFFSET + IEEE80211_TKIP_TX_MIC_OFFSET;
                 ieee80211_michael_mic(tx_mic, complete_buffer, size_wo_mic,
-                        complete_buffer + size_wo_mic);
-        }
-        
-        ieee80211_dev->ops->tx_handler(ieee80211_dev, 
-                complete_buffer, 
-                complete_size);
+                    complete_buffer + size_wo_mic);
+        }
+        
+        ieee80211_dev->ops->tx_handler(ieee80211_dev,
+            complete_buffer, complete_size);
         
         free(complete_buffer);
 }
 
-/**
- * Fill out IEEE 802.11 device functions implementations.
- * 
- * @param ieee80211_dev IEEE 802.11 device.
- * @param ieee80211_ops Callbacks implementation.
+/** Fill out IEEE 802.11 device functions implementations.
+ *
+ * @param ieee80211_dev   IEEE 802.11 device.
+ * @param ieee80211_ops   Callbacks implementation.
  * @param ieee80211_iface Interface functions implementation.
- * @param nic_iface NIC interface functions implementation.
- * @param nic_dev_ops NIC device functions implementation.
- * 
- * @return EINVAL when missing pointer to ieee80211_ops or ieee80211_iface,
- * otherwise EOK.
- */
-static int ieee80211_implement(ieee80211_dev_t *ieee80211_dev, 
-        ieee80211_ops_t *ieee80211_ops, ieee80211_iface_t *ieee80211_iface,
-        nic_iface_t *nic_iface, ddf_dev_ops_t *nic_dev_ops)
-{
-        if(ieee80211_ops) {
-                if(!ieee80211_ops->start)
+ * @param nic_iface       NIC interface functions implementation.
+ * @param nic_dev_ops     NIC device functions implementation.
+ *
+ * @return EINVAL when missing pointer to ieee80211_ops
+ *         or ieee80211_iface, otherwise EOK.
+ *
+ */
+static int ieee80211_implement(ieee80211_dev_t *ieee80211_dev,
+    ieee80211_ops_t *ieee80211_ops, ieee80211_iface_t *ieee80211_iface,
+    nic_iface_t *nic_iface, ddf_dev_ops_t *nic_dev_ops)
+{
+        if (ieee80211_ops) {
+                if (!ieee80211_ops->start)
                         ieee80211_ops->start = ieee80211_start_impl;
-
-                if(!ieee80211_ops->tx_handler)
+                
+                if (!ieee80211_ops->tx_handler)
                         ieee80211_ops->tx_handler = ieee80211_tx_handler_impl;
-
-                if(!ieee80211_ops->set_freq)
+                
+                if (!ieee80211_ops->set_freq)
                         ieee80211_ops->set_freq = ieee80211_set_freq_impl;
                 
-                if(!ieee80211_ops->bssid_change)
-                        ieee80211_ops->bssid_change = 
-                                ieee80211_bssid_change_impl;
-                
-                if(!ieee80211_ops->key_config)
+                if (!ieee80211_ops->bssid_change)
+                        ieee80211_ops->bssid_change = ieee80211_bssid_change_impl;
+                
+                if (!ieee80211_ops->key_config)
                         ieee80211_ops->key_config = ieee80211_key_config_impl;
                 
-                if(!ieee80211_ops->scan)
+                if (!ieee80211_ops->scan)
                         ieee80211_ops->scan = ieee80211_scan_impl;
-        } else {
+        } else
                 return EINVAL;
-        }
         
         ieee80211_dev->ops = ieee80211_ops;
         
-        if(ieee80211_iface) {
-                if(nic_dev_ops)
+        if (ieee80211_iface) {
+                if (nic_dev_ops)
                         if (!nic_dev_ops->interfaces[IEEE80211_DEV_IFACE])
-                                nic_dev_ops->interfaces[IEEE80211_DEV_IFACE] = 
-                                        ieee80211_iface;
-                
-                if(!ieee80211_iface->get_scan_results)
-                        ieee80211_iface->get_scan_results = 
-                                ieee80211_get_scan_results_impl;
-                
-                if(!ieee80211_iface->connect)
+                                nic_dev_ops->interfaces[IEEE80211_DEV_IFACE] =
+                                    ieee80211_iface;
+                
+                if (!ieee80211_iface->get_scan_results)
+                        ieee80211_iface->get_scan_results =
+                            ieee80211_get_scan_results_impl;
+                
+                if (!ieee80211_iface->connect)
                         ieee80211_iface->connect = ieee80211_connect_impl;
                 
-                if(!ieee80211_iface->disconnect)
+                if (!ieee80211_iface->disconnect)
                         ieee80211_iface->disconnect = ieee80211_disconnect_impl;
-        } else {
+        } else
                 return EINVAL;
-        }
-        
-        if(nic_dev_ops) {
-                if(!nic_dev_ops->open)
+        
+        if (nic_dev_ops) {
+                if (!nic_dev_ops->open)
                         nic_dev_ops->open = ieee80211_open;
-        } else {
+        } else
                 return EINVAL;
-        }
         
         ieee80211_dev->iface = ieee80211_iface;
@@ -721 +713 @@
 }
 
-/**
- * Allocate IEEE802.11 device structure.
- * 
+/** Allocate IEEE802.11 device structure.
+ *
  * @return Pointer to allocated IEEE802.11 device structure.
+ *
  */
 ieee80211_dev_t *ieee80211_device_create()
@@ -731 +723 @@
 }
 
-/**
- * Initialize an IEEE802.11 framework structure.
- * 
+/** Initialize an IEEE802.11 framework structure.
+ *
  * @param ieee80211_dev Device structure to initialize.
- * @param ddf_dev Pointer to backing DDF device structure.
- * 
+ * @param ddf_dev       Pointer to backing DDF device structure.
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 int ieee80211_device_init(ieee80211_dev_t *ieee80211_dev, ddf_dev_t *ddf_dev)
@@ -748 +740 @@
         ieee80211_dev->current_op_mode = IEEE80211_OPMODE_STATION;
         ieee80211_dev->current_auth_phase = IEEE80211_AUTH_DISCONNECTED;
-        memcpy(ieee80211_dev->bssid_mask.address, ieee80211_broadcast_mac_addr, 
-                ETH_ADDR);
+        
+        memcpy(ieee80211_dev->bssid_mask.address, ieee80211_broadcast_mac_addr,
+            ETH_ADDR);
         
         ieee80211_scan_result_list_init(&ieee80211_dev->ap_list);
@@ -759 +752 @@
         /* Bind NIC to device */
         nic_t *nic = nic_create_and_bind(ddf_dev);
-        if (!nic) {
+        if (!nic)
                 return ENOMEM;
-        }
         
         nic_set_specific(nic, ieee80211_dev);
@@ -768 +760 @@
 }
 
-/**
- * IEEE802.11 WiFi framework initialization.
- * 
- * @param ieee80211_dev Device structure to initialize.
- * @param ieee80211_ops Structure with implemented IEEE802.11 device operations.
- * @param ieee80211_iface Structure with implemented IEEE802.11 interface 
- * operations.
- * 
+/** IEEE802.11 WiFi framework initialization.
+ *
+ * @param ieee80211_dev   Device structure to initialize.
+ * @param ieee80211_ops   Structure with implemented IEEE802.11
+ *                        device operations.
+ * @param ieee80211_iface Structure with implemented IEEE802.11
+ *                        interface operations.
+ *
  * @return EOK if succeed, negative error code otherwise.
- */
-int ieee80211_init(ieee80211_dev_t *ieee80211_dev, 
-        ieee80211_ops_t *ieee80211_ops, ieee80211_iface_t *ieee80211_iface,
-        nic_iface_t *ieee80211_nic_iface, ddf_dev_ops_t *ieee80211_nic_dev_ops)
-{
-        int rc = ieee80211_implement(ieee80211_dev, 
-                ieee80211_ops, ieee80211_iface,
-                ieee80211_nic_iface, ieee80211_nic_dev_ops);
-        if(rc != EOK) {
+ *
+ */
+int ieee80211_init(ieee80211_dev_t *ieee80211_dev,
+    ieee80211_ops_t *ieee80211_ops, ieee80211_iface_t *ieee80211_iface,
+    nic_iface_t *ieee80211_nic_iface, ddf_dev_ops_t *ieee80211_nic_dev_ops)
+{
+        int rc = ieee80211_implement(ieee80211_dev,
+            ieee80211_ops, ieee80211_iface,
+            ieee80211_nic_iface, ieee80211_nic_dev_ops);
+        if (rc != EOK)
                 return rc;
-        }
         
         nic_t *nic = nic_get_from_ddf_dev(ieee80211_dev->ddf_dev);
         
-        /** TODO: Set NIC handlers here. */
+        /* TODO: Set NIC handlers here. */
         nic_set_send_frame_handler(nic, ieee80211_send_frame);
         
-        ddf_fun_t *fun = ddf_fun_create(ieee80211_dev->ddf_dev, fun_exposed, 
-                "port0");
-        if (fun == NULL) {
+        ddf_fun_t *fun = ddf_fun_create(ieee80211_dev->ddf_dev, fun_exposed,
+            "port0");
+        if (fun == NULL)
                 return EINVAL;
-        }
         
         nic_set_ddf_fun(nic, fun);
@@ -808 +799 @@
                 return rc;
         }
+        
         rc = ddf_fun_add_to_category(fun, DEVICE_CATEGORY_NIC);
         if (rc != EOK) {
@@ -813 +805 @@
                 return rc;
         }
+        
         rc = ddf_fun_add_to_category(fun, DEVICE_CATEGORY_IEEE80211);
         if (rc != EOK) {
@@ -822 +815 @@
 }
 
-/**
- * Convert frequency value to channel number.
- * 
+/** Convert frequency value to channel number.
+ *
  * @param freq IEEE 802.11 operating frequency.
- * 
+ *
  * @return Operating channel number.
+ *
  */
 static uint8_t ieee80211_freq_to_channel(uint16_t freq)
@@ -835 +828 @@
 
 static void ieee80211_prepare_ie_header(void **ie_header,
-        uint8_t id, uint8_t length, void *data)
+    uint8_t id, uint8_t length, void *data)
 {
         ieee80211_ie_header_t *header =
-                (ieee80211_ie_header_t *) *ie_header;
+            (ieee80211_ie_header_t *) *ie_header;
         
         header->element_id = id;
@@ -845 +838 @@
         memcpy(*ie_header + sizeof(ieee80211_ie_header_t), data, length);
         
-        *ie_header = (void *) ((void *) header + 
-                sizeof(ieee80211_ie_header_t) + length);
-}
-
-/**
- * Probe request implementation.
- * 
+        *ie_header = (void *) ((void *) header +
+            sizeof(ieee80211_ie_header_t) + length);
+}
+
+/** Probe request implementation.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param ssid Probing SSID or NULL if broadcast.
- * 
+ * @param ssid          Probing SSID or NULL if broadcast.
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 int ieee80211_probe_request(ieee80211_dev_t *ieee80211_dev, char *ssid)
@@ -866 +859 @@
         size_t channel_data_size = 1;
         
-        uint8_t channel = 
-                ieee80211_freq_to_channel(ieee80211_dev->current_freq);
-        
-        /* 4 headers - (ssid, rates, ext rates, current channel) and their data
-         * lengths. 
+        uint8_t channel =
+            ieee80211_freq_to_channel(ieee80211_dev->current_freq);
+        
+        /*
+         * 4 headers - (ssid, rates, ext rates, current channel)
+         * and their data lengths.
          */
-        size_t payload_size = 
-                sizeof(ieee80211_ie_header_t) * 4 +
-                ssid_data_size + 
-                IEEE80211_DATA_RATES_SIZE + IEEE80211_EXT_DATA_RATES_SIZE + 
-                channel_data_size;
+        size_t payload_size =
+            sizeof(ieee80211_ie_header_t) * 4 +
+            ssid_data_size +
+            IEEE80211_DATA_RATES_SIZE + IEEE80211_EXT_DATA_RATES_SIZE +
+            channel_data_size;
         
         size_t buffer_size = sizeof(ieee80211_mgmt_header_t) + payload_size;
         void *buffer = malloc(buffer_size);
+        if (!buffer)
+                return ENOMEM;
+        
         memset(buffer, 0, buffer_size);
         
-        ieee80211_mgmt_header_t *mgmt_header = 
-                (ieee80211_mgmt_header_t *) buffer;
-        
-        mgmt_header->frame_ctrl = host2uint16_t_le(
-                IEEE80211_MGMT_FRAME | 
-                IEEE80211_MGMT_PROBE_REQ_FRAME
-                );
+        ieee80211_mgmt_header_t *mgmt_header =
+            (ieee80211_mgmt_header_t *) buffer;
+        
+        mgmt_header->frame_ctrl =
+            host2uint16_t_le(IEEE80211_MGMT_FRAME |
+            IEEE80211_MGMT_PROBE_REQ_FRAME);
         memcpy(mgmt_header->dest_addr, ieee80211_broadcast_mac_addr, ETH_ADDR);
         memcpy(mgmt_header->src_addr, nic_address.address, ETH_ADDR);
         memcpy(mgmt_header->bssid, ieee80211_broadcast_mac_addr, ETH_ADDR);
-        mgmt_header->seq_ctrl = 
-                host2uint16_t_le(ieee80211_get_sequence_number(ieee80211_dev));
+        mgmt_header->seq_ctrl =
+            host2uint16_t_le(ieee80211_get_sequence_number(ieee80211_dev));
         
         /* Jump to payload. */
         void *it = (void *) buffer + sizeof(ieee80211_mgmt_header_t);
-        ieee80211_prepare_ie_header(&it, IEEE80211_SSID_IE, ssid_data_size, 
-                (void *) ssid);
-        ieee80211_prepare_ie_header(&it, IEEE80211_RATES_IE, 
-                IEEE80211_DATA_RATES_SIZE, 
-                (void *) &ieee80211bg_data_rates);
-        ieee80211_prepare_ie_header(&it, IEEE80211_EXT_RATES_IE, 
-                IEEE80211_EXT_DATA_RATES_SIZE, 
-                (void *) &ieee80211bg_data_rates[IEEE80211_DATA_RATES_SIZE]);
-        ieee80211_prepare_ie_header(&it, IEEE80211_CHANNEL_IE, 
-                channel_data_size, (void *) &channel);
+        ieee80211_prepare_ie_header(&it, IEEE80211_SSID_IE, ssid_data_size,
+            (void *) ssid);
+        ieee80211_prepare_ie_header(&it, IEEE80211_RATES_IE,
+            IEEE80211_DATA_RATES_SIZE, (void *) &ieee80211bg_data_rates);
+        ieee80211_prepare_ie_header(&it, IEEE80211_EXT_RATES_IE,
+            IEEE80211_EXT_DATA_RATES_SIZE,
+            (void *) &ieee80211bg_data_rates[IEEE80211_DATA_RATES_SIZE]);
+        ieee80211_prepare_ie_header(&it, IEEE80211_CHANNEL_IE,
+            channel_data_size, (void *) &channel);
         
         ieee80211_dev->ops->tx_handler(ieee80211_dev, buffer, buffer_size);
@@ -915 +910 @@
 }
 
-/**
- * IEEE 802.11 authentication implementation.
- * 
+/** IEEE 802.11 authentication implementation.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * 
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 int ieee80211_authenticate(ieee80211_dev_t *ieee80211_dev)
@@ -929 +924 @@
         
         ieee80211_scan_result_t *auth_data =
-                &ieee80211_dev->bssid_info.res_link->scan_result;
-        
-        size_t buffer_size = sizeof(ieee80211_mgmt_header_t) + 
-                sizeof(ieee80211_auth_body_t);
+            &ieee80211_dev->bssid_info.res_link->scan_result;
+        
+        size_t buffer_size = sizeof(ieee80211_mgmt_header_t) +
+            sizeof(ieee80211_auth_body_t);
         
         void *buffer = malloc(buffer_size);
+        if (!buffer)
+                return ENOMEM;
+        
         memset(buffer, 0, buffer_size);
         
-        ieee80211_mgmt_header_t *mgmt_header = 
-                (ieee80211_mgmt_header_t *) buffer;
-        
-        mgmt_header->frame_ctrl = host2uint16_t_le(
-                IEEE80211_MGMT_FRAME | 
-                IEEE80211_MGMT_AUTH_FRAME
-                );
+        ieee80211_mgmt_header_t *mgmt_header =
+            (ieee80211_mgmt_header_t *) buffer;
+        
+        mgmt_header->frame_ctrl =
+            host2uint16_t_le(IEEE80211_MGMT_FRAME |
+            IEEE80211_MGMT_AUTH_FRAME);
         memcpy(mgmt_header->dest_addr, auth_data->bssid.address, ETH_ADDR);
         memcpy(mgmt_header->src_addr, nic_address.address, ETH_ADDR);
@@ -949 +946 @@
         
         ieee80211_auth_body_t *auth_body =
-                (ieee80211_auth_body_t *) 
-                (buffer + sizeof(ieee80211_mgmt_header_t));
+            (ieee80211_auth_body_t *)
+            (buffer + sizeof(ieee80211_mgmt_header_t));
         auth_body->auth_alg = host2uint16_t_le(0);
         auth_body->auth_trans_no = host2uint16_t_le(1);
@@ -961 +958 @@
 }
 
-/** 
- * IEEE 802.11 association implementation.
- * 
+/** IEEE 802.11 association implementation.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param password Passphrase to be used in encrypted communication or NULL
- * for open networks.
- * 
+ * @param password      Passphrase to be used in encrypted communication
+ *                      or NULL for open networks.
+ *
  * @return EOK if succeed, negative error code otherwise.
- */
-int ieee80211_associate(ieee80211_dev_t *ieee80211_dev, char *password) 
+ *
+ */
+int ieee80211_associate(ieee80211_dev_t *ieee80211_dev, char *password)
 {
         nic_t *nic = nic_get_from_ddf_dev(ieee80211_dev->ddf_dev);
@@ -977 +974 @@
         
         ieee80211_scan_result_link_t *auth_link =
-                ieee80211_dev->bssid_info.res_link;
+            ieee80211_dev->bssid_info.res_link;
         
         ieee80211_scan_result_t *auth_data = &auth_link->scan_result;
@@ -983 +980 @@
         size_t ssid_data_size = str_size(auth_data->ssid);
         
-        size_t payload_size = 
-                sizeof(ieee80211_ie_header_t) * 3 +
-                ssid_data_size + 
-                IEEE80211_DATA_RATES_SIZE + 
-                IEEE80211_EXT_DATA_RATES_SIZE;
-        
-        size_t buffer_size = 
-                sizeof(ieee80211_mgmt_header_t) + 
-                sizeof(ieee80211_assoc_req_body_t) +
-                payload_size;
-        
-        if(auth_data->security.type == IEEE80211_SECURITY_WPA || 
-                auth_data->security.type == IEEE80211_SECURITY_WPA2) {
+        size_t payload_size =
+            sizeof(ieee80211_ie_header_t) * 3 +
+            ssid_data_size +
+            IEEE80211_DATA_RATES_SIZE +
+            IEEE80211_EXT_DATA_RATES_SIZE;
+        
+        size_t buffer_size =
+            sizeof(ieee80211_mgmt_header_t) +
+            sizeof(ieee80211_assoc_req_body_t) +
+            payload_size;
+        
+        if ((auth_data->security.type == IEEE80211_SECURITY_WPA) ||
+            (auth_data->security.type == IEEE80211_SECURITY_WPA2))
                 buffer_size += auth_link->auth_ie_len;
-        }
         
         void *buffer = malloc(buffer_size);
+        if (!buffer)
+                return ENOMEM;
+        
         memset(buffer, 0, buffer_size);
         
-        ieee80211_mgmt_header_t *mgmt_header = 
-                (ieee80211_mgmt_header_t *) buffer;
-        
-        mgmt_header->frame_ctrl = host2uint16_t_le(
-                IEEE80211_MGMT_FRAME | 
-                IEEE80211_MGMT_ASSOC_REQ_FRAME
-                );
+        ieee80211_mgmt_header_t *mgmt_header =
+            (ieee80211_mgmt_header_t *) buffer;
+        
+        mgmt_header->frame_ctrl =
+            host2uint16_t_le(IEEE80211_MGMT_FRAME |
+            IEEE80211_MGMT_ASSOC_REQ_FRAME);
         memcpy(mgmt_header->dest_addr, auth_data->bssid.address, ETH_ADDR);
         memcpy(mgmt_header->src_addr, nic_address.address, ETH_ADDR);
@@ -1014 +1012 @@
         
         ieee80211_assoc_req_body_t *assoc_body =
-                (ieee80211_assoc_req_body_t *) 
-                (buffer + sizeof(ieee80211_mgmt_header_t));
+            (ieee80211_assoc_req_body_t *)
+            (buffer + sizeof(ieee80211_mgmt_header_t));
         assoc_body->listen_interval = host2uint16_t_le(1);
         
         /* Jump to payload. */
         void *it = buffer + sizeof(ieee80211_mgmt_header_t) +
-                sizeof(ieee80211_assoc_req_body_t);
-        ieee80211_prepare_ie_header(&it, IEEE80211_SSID_IE, 
-                ssid_data_size, (void *) auth_data->ssid);
-        ieee80211_prepare_ie_header(&it, IEEE80211_RATES_IE, 
-                IEEE80211_DATA_RATES_SIZE, 
-                (void *) &ieee80211bg_data_rates);
-        ieee80211_prepare_ie_header(&it, IEEE80211_EXT_RATES_IE, 
-                IEEE80211_EXT_DATA_RATES_SIZE, 
-                (void *) &ieee80211bg_data_rates[IEEE80211_DATA_RATES_SIZE]);
-        
-        if(auth_data->security.type != IEEE80211_SECURITY_OPEN) {
+            sizeof(ieee80211_assoc_req_body_t);
+        ieee80211_prepare_ie_header(&it, IEEE80211_SSID_IE,
+            ssid_data_size, (void *) auth_data->ssid);
+        ieee80211_prepare_ie_header(&it, IEEE80211_RATES_IE,
+            IEEE80211_DATA_RATES_SIZE, (void *) &ieee80211bg_data_rates);
+        ieee80211_prepare_ie_header(&it, IEEE80211_EXT_RATES_IE,
+            IEEE80211_EXT_DATA_RATES_SIZE,
+            (void *) &ieee80211bg_data_rates[IEEE80211_DATA_RATES_SIZE]);
+        
+        if (auth_data->security.type != IEEE80211_SECURITY_OPEN)
                 assoc_body->capability |= host2uint16_t_le(CAP_SECURITY);
-        }
-        
-        if(auth_data->security.type == IEEE80211_SECURITY_WPA || 
-                auth_data->security.type == IEEE80211_SECURITY_WPA2) {
-                memcpy(it, auth_link->auth_ie,  auth_link->auth_ie_len);
-        }
+        
+        if ((auth_data->security.type == IEEE80211_SECURITY_WPA) ||
+            (auth_data->security.type == IEEE80211_SECURITY_WPA2))
+                memcpy(it, auth_link->auth_ie, auth_link->auth_ie_len);
         
         ieee80211_dev->ops->tx_handler(ieee80211_dev, buffer, buffer_size);
         
-        /* 
-         * Save password to be used in eventual authentication handshake. 
+        /*
+         * Save password to be used in eventual authentication handshake.
          */
         memset(ieee80211_dev->bssid_info.password, 0, IEEE80211_MAX_PASSW_LEN);
-        memcpy(ieee80211_dev->bssid_info.password, password, 
-                str_size(password));
+        memcpy(ieee80211_dev->bssid_info.password, password,
+            str_size(password));
         
         free(buffer);
@@ -1053 +1048 @@
 }
 
-/** 
- * IEEE 802.11 deauthentication implementation.
- * 
+/** IEEE 802.11 deauthentication implementation.
+ *
  * Note: Expecting locked results_mutex or scan_mutex.
- * 
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * 
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 int ieee80211_deauthenticate(ieee80211_dev_t *ieee80211_dev)
 {
         ieee80211_scan_result_t *auth_data =
-                &ieee80211_dev->bssid_info.res_link->scan_result;
+            &ieee80211_dev->bssid_info.res_link->scan_result;
         
         nic_t *nic = nic_get_from_ddf_dev(ieee80211_dev->ddf_dev);
@@ -1071 +1066 @@
         nic_query_address(nic, &nic_address);
         
-        size_t buffer_size = sizeof(ieee80211_mgmt_header_t) + 
-                sizeof(ieee80211_deauth_body_t);
+        size_t buffer_size = sizeof(ieee80211_mgmt_header_t) +
+            sizeof(ieee80211_deauth_body_t);
+        
         void *buffer = malloc(buffer_size);
+        if (!buffer)
+                return ENOMEM;
+        
         memset(buffer, 0, buffer_size);
         
-        ieee80211_mgmt_header_t *mgmt_header = 
-                (ieee80211_mgmt_header_t *) buffer;
-        
-        mgmt_header->frame_ctrl = host2uint16_t_le(
-                IEEE80211_MGMT_FRAME | 
-                IEEE80211_MGMT_DEAUTH_FRAME
-                );
+        ieee80211_mgmt_header_t *mgmt_header =
+            (ieee80211_mgmt_header_t *) buffer;
+        
+        mgmt_header->frame_ctrl =
+            host2uint16_t_le(IEEE80211_MGMT_FRAME |
+            IEEE80211_MGMT_DEAUTH_FRAME);
         memcpy(mgmt_header->dest_addr, auth_data->bssid.address, ETH_ADDR);
         memcpy(mgmt_header->src_addr, nic_address.address, ETH_ADDR);
@@ -1094 +1092 @@
         ieee80211_dev->ops->bssid_change(ieee80211_dev, false);
         
-        if(ieee80211_query_using_key(ieee80211_dev))
+        if (ieee80211_query_using_key(ieee80211_dev))
                 ieee80211_dev->ops->key_config(ieee80211_dev, NULL, false);
         
@@ -1103 +1101 @@
 
 static void ieee80211_process_auth_info(ieee80211_scan_result_link_t *ap_data,
-        void *buffer)
+    void *buffer)
 {
         uint8_t *it = (uint8_t *) buffer;
         
         uint16_t *version = (uint16_t *) it;
-        if(uint16_t_le2host(*version) != 0x1) {
+        if (uint16_t_le2host(*version) != 0x1) {
                 ap_data->scan_result.security.type = -1;
                 return;
@@ -1115 +1113 @@
         it += sizeof(uint16_t);
         
-        uint32_t group_cipher = *(it+3);
-        switch(group_cipher) {
-                case IEEE80211_AUTH_CIPHER_TKIP:
-                        ap_data->scan_result.security.group_alg =
-                                IEEE80211_SECURITY_SUITE_TKIP;
-                        break;
-                case IEEE80211_AUTH_CIPHER_CCMP:
-                        ap_data->scan_result.security.group_alg =
-                                IEEE80211_SECURITY_SUITE_CCMP;
-                        break;
-                default:
-                        ap_data->scan_result.security.group_alg = -1;
-        }
-        
-        it += 4*sizeof(uint8_t);
+        uint32_t group_cipher = *(it + 3);
+        switch (group_cipher) {
+        case IEEE80211_AUTH_CIPHER_TKIP:
+                ap_data->scan_result.security.group_alg =
+                    IEEE80211_SECURITY_SUITE_TKIP;
+                break;
+        case IEEE80211_AUTH_CIPHER_CCMP:
+                ap_data->scan_result.security.group_alg =
+                    IEEE80211_SECURITY_SUITE_CCMP;
+                break;
+        default:
+                ap_data->scan_result.security.group_alg = -1;
+        }
+        
+        it += 4 * sizeof(uint8_t);
         
         uint16_t *pairwise_count = (uint16_t *) it;
-        uint32_t pairwise_cipher = *(it+sizeof(uint16_t)+3);
-        switch(pairwise_cipher) {
-                case IEEE80211_AUTH_CIPHER_TKIP:
-                        ap_data->scan_result.security.pair_alg =
-                                IEEE80211_SECURITY_SUITE_TKIP;
-                        break;
-                case IEEE80211_AUTH_CIPHER_CCMP:
-                        ap_data->scan_result.security.pair_alg =
-                                IEEE80211_SECURITY_SUITE_CCMP;
-                        break;
-                default:
-                        ap_data->scan_result.security.pair_alg = -1;
-        }
-        
-        it += 2*sizeof(uint16_t) + 
-                uint16_t_le2host(*pairwise_count)*sizeof(uint32_t);
-        
-        uint32_t auth_suite = *(it+3);
-        switch(auth_suite) {
-                case IEEE80211_AUTH_AKM_PSK:
-                        ap_data->scan_result.security.auth =
-                                IEEE80211_SECURITY_AUTH_PSK;
-                        break;
-                case IEEE80211_AUTH_AKM_8021X:
-                        ap_data->scan_result.security.auth =
-                                IEEE80211_SECURITY_AUTH_8021X;
-                        break;
-                default:
-                        ap_data->scan_result.security.auth = -1;
+        uint32_t pairwise_cipher = *(it + sizeof(uint16_t) + 3);
+        switch (pairwise_cipher) {
+        case IEEE80211_AUTH_CIPHER_TKIP:
+                ap_data->scan_result.security.pair_alg =
+                    IEEE80211_SECURITY_SUITE_TKIP;
+                break;
+        case IEEE80211_AUTH_CIPHER_CCMP:
+                ap_data->scan_result.security.pair_alg =
+                    IEEE80211_SECURITY_SUITE_CCMP;
+                break;
+        default:
+                ap_data->scan_result.security.pair_alg = -1;
+        }
+        
+        it += 2 * sizeof(uint16_t) +
+            uint16_t_le2host(*pairwise_count) * sizeof(uint32_t);
+        
+        uint32_t auth_suite = *(it + 3);
+        switch (auth_suite) {
+        case IEEE80211_AUTH_AKM_PSK:
+                ap_data->scan_result.security.auth =
+                    IEEE80211_SECURITY_AUTH_PSK;
+                break;
+        case IEEE80211_AUTH_AKM_8021X:
+                ap_data->scan_result.security.auth =
+                    IEEE80211_SECURITY_AUTH_8021X;
+                break;
+        default:
+                ap_data->scan_result.security.auth = -1;
         }
 }
 
 static void copy_auth_ie(ieee80211_ie_header_t *ie_header,
-        ieee80211_scan_result_link_t *ap_data, void *it)
-{
-        ap_data->auth_ie_len = ie_header->length + 
-                sizeof(ieee80211_ie_header_t);
+    ieee80211_scan_result_link_t *ap_data, void *it)
+{
+        ap_data->auth_ie_len = ie_header->length +
+            sizeof(ieee80211_ie_header_t);
         
         memcpy(ap_data->auth_ie, it, ap_data->auth_ie_len);
@@ -1174 +1172 @@
 
 static uint8_t *ieee80211_process_ies(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_scan_result_link_t *ap_data, void *buffer, size_t buffer_size)
+    ieee80211_scan_result_link_t *ap_data, void *buffer, size_t buffer_size)
 {
         void *it = buffer;
-        while((it + sizeof(ieee80211_ie_header_t)) < buffer + buffer_size) {
-                ieee80211_ie_header_t *ie_header = 
-                        (ieee80211_ie_header_t *) it;
+        while ((it + sizeof(ieee80211_ie_header_t)) < buffer + buffer_size) {
+                ieee80211_ie_header_t *ie_header =
+                    (ieee80211_ie_header_t *) it;
                 uint8_t *channel;
                 uint32_t oui;
-                switch(ie_header->element_id) {
-                        case IEEE80211_CHANNEL_IE:
-                                if(!ap_data)
+                
+                switch (ie_header->element_id) {
+                case IEEE80211_CHANNEL_IE:
+                        if (!ap_data)
+                                break;
+                        
+                        channel = (uint8_t *)
+                            (it + sizeof(ieee80211_ie_header_t));
+                        ap_data->scan_result.channel = *channel;
+                        break;
+                case IEEE80211_RSN_IE:
+                        if (!ap_data)
+                                break;
+                        
+                        ap_data->scan_result.security.type =
+                            IEEE80211_SECURITY_WPA2;
+                        ieee80211_process_auth_info(ap_data,
+                            it + sizeof(ieee80211_ie_header_t));
+                        copy_auth_ie(ie_header, ap_data, it);
+                        break;
+                case IEEE80211_VENDOR_IE:
+                        oui = uint32be_from_seq(it +
+                            sizeof(ieee80211_ie_header_t));
+                        
+                        if (oui == WPA_OUI) {
+                                if (!ap_data)
                                         break;
-                                channel = (uint8_t *) 
-                                        (it + sizeof(ieee80211_ie_header_t));
-                                ap_data->scan_result.channel = *channel;
-                                break;
-                        case IEEE80211_RSN_IE:
-                                if(!ap_data)
+                                
+                                /* Prefering WPA2. */
+                                if (ap_data->scan_result.security.type ==
+                                    IEEE80211_SECURITY_WPA2)
                                         break;
+                                
                                 ap_data->scan_result.security.type =
-                                        IEEE80211_SECURITY_WPA2;
-                                ieee80211_process_auth_info(ap_data, 
-                                        it + sizeof(ieee80211_ie_header_t));
+                                    IEEE80211_SECURITY_WPA;
+                                
+                                ieee80211_process_auth_info(ap_data,
+                                    it + sizeof(ieee80211_ie_header_t) +
+                                    sizeof(uint32_t));
                                 copy_auth_ie(ie_header, ap_data, it);
-                                break;
-                        case IEEE80211_VENDOR_IE:
-                                oui = uint32be_from_seq(it + 
-                                        sizeof(ieee80211_ie_header_t));
-                                if(oui == WPA_OUI) {
-                                        if(!ap_data)
-                                                break;
-                                        /* Prefering WPA2. */
-                                        if(ap_data->scan_result.security.type ==
-                                                IEEE80211_SECURITY_WPA2) {
-                                                break;
-                                        }
-                                        ap_data->scan_result.security.type =
-                                                IEEE80211_SECURITY_WPA;
-                                        ieee80211_process_auth_info(ap_data,
-                                                it + 
-                                                sizeof(ieee80211_ie_header_t) +
-                                                sizeof(uint32_t));
-                                        copy_auth_ie(ie_header, ap_data, it);
-                                } else if(oui == GTK_OUI) {
-                                        return it + 
-                                                sizeof(ieee80211_ie_header_t) +
-                                                sizeof(uint32_t);
-                                }
+                        } else if (oui == GTK_OUI) {
+                                return it +
+                                    sizeof(ieee80211_ie_header_t) +
+                                    sizeof(uint32_t);
+                        }
                 }
+                
                 it += sizeof(ieee80211_ie_header_t) + ie_header->length;
         }
@@ -1229 +1233 @@
 }
 
-/**
- * Process probe response and store results.
- * 
+/** Process probe response and store results.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param mgmt_header Pointer to start of management frame header.
- * 
+ * @param mgmt_header   Pointer to start of management frame header.
+ *
  * @return EOK if succeed, negative error code otherwise.
- */
-static int ieee80211_process_probe_response(ieee80211_dev_t *ieee80211_dev, 
-        ieee80211_mgmt_header_t *mgmt_header, size_t buffer_size)
-{
-        ieee80211_beacon_start_t *beacon_body = (ieee80211_beacon_start_t *) 
-                ((void *)mgmt_header + sizeof(ieee80211_mgmt_header_t));
-        
-        ieee80211_ie_header_t *ssid_ie_header = (ieee80211_ie_header_t *) 
-                ((void *)beacon_body + sizeof(ieee80211_beacon_start_t));
+ *
+ */
+static int ieee80211_process_probe_response(ieee80211_dev_t *ieee80211_dev,
+    ieee80211_mgmt_header_t *mgmt_header, size_t buffer_size)
+{
+        ieee80211_beacon_start_t *beacon_body = (ieee80211_beacon_start_t *)
+            ((void *) mgmt_header + sizeof(ieee80211_mgmt_header_t));
+        
+        ieee80211_ie_header_t *ssid_ie_header = (ieee80211_ie_header_t *)
+            ((void *) beacon_body + sizeof(ieee80211_beacon_start_t));
         
         /* Not empty SSID. */
-        if(ssid_ie_header->length > 0) {
+        if (ssid_ie_header->length > 0) {
                 ieee80211_scan_result_list_t *result_list =
-                        &ieee80211_dev->ap_list;
-                
-                uint8_t *ssid_start = (uint8_t *) ((void *)ssid_ie_header + 
-                        sizeof(ieee80211_ie_header_t));
+                    &ieee80211_dev->ap_list;
+                
+                uint8_t *ssid_start = (uint8_t *) ((void *) ssid_ie_header +
+                    sizeof(ieee80211_ie_header_t));
                 char ssid[IEEE80211_MAX_SSID_LENGTH];
+                
                 memcpy(ssid, ssid_start, ssid_ie_header->length);
                 ssid[ssid_ie_header->length] = '\0';
@@ -1259 +1264 @@
                 /* Check whether SSID is already in results. */
                 ieee80211_scan_result_list_foreach(*result_list, result) {
-                        if(!str_cmp(ssid, result->scan_result.ssid)) {
+                        if (!str_cmp(ssid, result->scan_result.ssid)) {
                                 result->last_beacon = time(NULL);
                                 return EOK;
@@ -1266 +1271 @@
                 
                 /* Results are full. */
-                if(result_list->size == IEEE80211_MAX_RESULTS_LENGTH - 1) {
+                if (result_list->size == IEEE80211_MAX_RESULTS_LENGTH - 1)
                         return EOK;
-                }
-                
-                ieee80211_scan_result_link_t *ap_data = 
-                        malloc(sizeof(ieee80211_scan_result_link_t));
+                
+                ieee80211_scan_result_link_t *ap_data =
+                    malloc(sizeof(ieee80211_scan_result_link_t));
+                if (!ap_data)
+                        return ENOMEM;
+                
                 memset(ap_data, 0, sizeof(ieee80211_scan_result_link_t));
                 link_initialize(&ap_data->link);
                 
-                memcpy(ap_data->scan_result.bssid.address, 
-                        mgmt_header->bssid, ETH_ADDR);
-                memcpy(ap_data->scan_result.ssid, ssid, 
-                        ssid_ie_header->length + 1);
-                
-                if(uint16_t_le2host(beacon_body->capability) & CAP_SECURITY) {
-                        ap_data->scan_result.security.type = 
-                                IEEE80211_SECURITY_WEP;
+                memcpy(ap_data->scan_result.bssid.address,
+                    mgmt_header->bssid, ETH_ADDR);
+                memcpy(ap_data->scan_result.ssid, ssid,
+                    ssid_ie_header->length + 1);
+                
+                if (uint16_t_le2host(beacon_body->capability) & CAP_SECURITY) {
+                        ap_data->scan_result.security.type =
+                            IEEE80211_SECURITY_WEP;
                 } else {
-                        ap_data->scan_result.security.type = 
-                                IEEE80211_SECURITY_OPEN;
+                        ap_data->scan_result.security.type =
+                            IEEE80211_SECURITY_OPEN;
                         ap_data->scan_result.security.auth = -1;
                         ap_data->scan_result.security.pair_alg = -1;
@@ -1292 +1299 @@
                 
                 void *rest_ies_start = ssid_start + ssid_ie_header->length;
-                size_t rest_buffer_size = 
-                        buffer_size - 
-                        sizeof(ieee80211_mgmt_header_t) -
-                        sizeof(ieee80211_beacon_start_t) -
-                        sizeof(ieee80211_ie_header_t) -
-                        ssid_ie_header->length;
-                
-                ieee80211_process_ies(ieee80211_dev, ap_data, rest_ies_start, 
-                        rest_buffer_size);
+                size_t rest_buffer_size =
+                    buffer_size -
+                    sizeof(ieee80211_mgmt_header_t) -
+                    sizeof(ieee80211_beacon_start_t) -
+                    sizeof(ieee80211_ie_header_t) -
+                    ssid_ie_header->length;
+                
+                ieee80211_process_ies(ieee80211_dev, ap_data, rest_ies_start,
+                    rest_buffer_size);
                 
                 ap_data->last_beacon = time(NULL);
@@ -1312 +1319 @@
 }
 
-/**
- * Process authentication response.
- * 
+/** Process authentication response.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param mgmt_header Pointer to start of management frame header.
- * 
+ * @param mgmt_header   Pointer to start of management frame header.
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 static int ieee80211_process_auth_response(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_mgmt_header_t *mgmt_header)
+    ieee80211_mgmt_header_t *mgmt_header)
 {
         ieee80211_auth_body_t *auth_body =
-                (ieee80211_auth_body_t *)
-                ((void *)mgmt_header + sizeof(ieee80211_mgmt_header_t));
-        
-        if(auth_body->status != 0) {
-                ieee80211_set_auth_phase(ieee80211_dev, 
-                        IEEE80211_AUTH_DISCONNECTED);
-        } else {
-                ieee80211_set_auth_phase(ieee80211_dev, 
-                        IEEE80211_AUTH_AUTHENTICATED);
-        }
+            (ieee80211_auth_body_t *)
+            ((void *) mgmt_header + sizeof(ieee80211_mgmt_header_t));
+        
+        if (auth_body->status != 0)
+                ieee80211_set_auth_phase(ieee80211_dev,
+                    IEEE80211_AUTH_DISCONNECTED);
+        else
+                ieee80211_set_auth_phase(ieee80211_dev,
+                    IEEE80211_AUTH_AUTHENTICATED);
         
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
@@ -1342 +1348 @@
 }
 
-/**
- * Process association response.
- * 
+/** Process association response.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param mgmt_header Pointer to start of management frame header.
- * 
+ * @param mgmt_header   Pointer to start of management frame header.
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 static int ieee80211_process_assoc_response(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_mgmt_header_t *mgmt_header)
+    ieee80211_mgmt_header_t *mgmt_header)
 {
         ieee80211_assoc_resp_body_t *assoc_resp =
-                (ieee80211_assoc_resp_body_t *) ((void *)mgmt_header +
-                sizeof(ieee80211_mgmt_header_t));
-        
-        if(assoc_resp->status != 0) {
-                ieee80211_set_auth_phase(ieee80211_dev, 
-                        IEEE80211_AUTH_DISCONNECTED);
-        } else {
-                ieee80211_dev->bssid_info.aid = 
-                        uint16_t_le2host(assoc_resp->aid);
-                ieee80211_set_auth_phase(ieee80211_dev, 
-                        IEEE80211_AUTH_ASSOCIATED);
+            (ieee80211_assoc_resp_body_t *) ((void *) mgmt_header +
+            sizeof(ieee80211_mgmt_header_t));
+        
+        if (assoc_resp->status != 0)
+                ieee80211_set_auth_phase(ieee80211_dev,
+                    IEEE80211_AUTH_DISCONNECTED);
+        else {
+                ieee80211_dev->bssid_info.aid =
+                    uint16_t_le2host(assoc_resp->aid);
+                ieee80211_set_auth_phase(ieee80211_dev,
+                    IEEE80211_AUTH_ASSOCIATED);
                 ieee80211_dev->ops->bssid_change(ieee80211_dev, true);
         }
@@ -1376 +1382 @@
 
 static int ieee80211_process_4way_handshake(ieee80211_dev_t *ieee80211_dev,
-        void *buffer, size_t buffer_size)
-{
-        ieee80211_eapol_key_frame_t *key_frame = 
-                (ieee80211_eapol_key_frame_t *) buffer;
-        
-        ieee80211_scan_result_link_t *auth_link = 
-                ieee80211_dev->bssid_info.res_link;
-
+    void *buffer, size_t buffer_size)
+{
+        ieee80211_eapol_key_frame_t *key_frame =
+            (ieee80211_eapol_key_frame_t *) buffer;
+        
+        ieee80211_scan_result_link_t *auth_link =
+            ieee80211_dev->bssid_info.res_link;
+        
         ieee80211_scan_result_t *auth_data = &auth_link->scan_result;
         
         /* We don't support 802.1X authentication yet. */
-        if(auth_data->security.auth == IEEE80211_AUTH_AKM_8021X) {
+        if (auth_data->security.auth == IEEE80211_AUTH_AKM_8021X)
                 return ENOTSUP;
-        }
         
         uint8_t *ptk = ieee80211_dev->bssid_info.ptk;
         uint8_t *gtk = ieee80211_dev->bssid_info.gtk;
         uint8_t gtk_id = 1;
-
+        
         bool handshake_done = false;
         
-        bool old_wpa = 
-                auth_data->security.type == IEEE80211_SECURITY_WPA;
+        bool old_wpa =
+            auth_data->security.type == IEEE80211_SECURITY_WPA;
         
         bool key_phase =
-                uint16_t_be2host(key_frame->key_info) & 
-                IEEE80211_EAPOL_KEY_KEYINFO_MIC;
-        
-        bool final_phase = 
-                uint16_t_be2host(key_frame->key_info) & 
-                IEEE80211_EAPOL_KEY_KEYINFO_SECURE;
-        
-        bool ccmp_used = 
-                auth_data->security.pair_alg == IEEE80211_SECURITY_SUITE_CCMP ||
-                auth_data->security.group_alg == IEEE80211_SECURITY_SUITE_CCMP;
+            uint16_t_be2host(key_frame->key_info) &
+            IEEE80211_EAPOL_KEY_KEYINFO_MIC;
+        
+        bool final_phase =
+            uint16_t_be2host(key_frame->key_info) &
+            IEEE80211_EAPOL_KEY_KEYINFO_SECURE;
+        
+        bool ccmp_used =
+            (auth_data->security.pair_alg == IEEE80211_SECURITY_SUITE_CCMP) ||
+            (auth_data->security.group_alg == IEEE80211_SECURITY_SUITE_CCMP);
         
         size_t ptk_key_length, gtk_key_length;
         hash_func_t mic_hash;
-        if(ccmp_used) {
+        if (ccmp_used)
                 mic_hash = HASH_SHA1;
-        } else {
+        else
                 mic_hash = HASH_MD5;
-        }
-        
-        if(auth_data->security.pair_alg == IEEE80211_SECURITY_SUITE_CCMP) {
+        
+        if (auth_data->security.pair_alg == IEEE80211_SECURITY_SUITE_CCMP)
                 ptk_key_length = IEEE80211_PTK_CCMP_LENGTH;
-        } else {
+        else
                 ptk_key_length = IEEE80211_PTK_TKIP_LENGTH;
-        }
-        
-        if(auth_data->security.group_alg == IEEE80211_SECURITY_SUITE_CCMP) {
+        
+        if (auth_data->security.group_alg == IEEE80211_SECURITY_SUITE_CCMP)
                 gtk_key_length = IEEE80211_GTK_CCMP_LENGTH;
-        } else {
+        else
                 gtk_key_length = IEEE80211_GTK_TKIP_LENGTH;
-        }
-        
-        size_t output_size = 
-                sizeof(eth_header_t) +
-                sizeof(ieee80211_eapol_key_frame_t);
-
-        if(!(uint16_t_be2host(key_frame->key_info) & 
-                IEEE80211_EAPOL_KEY_KEYINFO_MIC)) {
+        
+        size_t output_size =
+            sizeof(eth_header_t) +
+            sizeof(ieee80211_eapol_key_frame_t);
+        
+        if (!(uint16_t_be2host(key_frame->key_info) &
+            IEEE80211_EAPOL_KEY_KEYINFO_MIC))
                 output_size += auth_link->auth_ie_len;
-        }
-
+        
         nic_t *nic = nic_get_from_ddf_dev(ieee80211_dev->ddf_dev);
         nic_address_t nic_address;
         nic_query_address(nic, &nic_address);
-
+        
         void *output_buffer = malloc(output_size);
+        if (!output_buffer)
+                return ENOMEM;
+        
         memset(output_buffer, 0, output_size);
-
+        
         /* Setup ethernet header. */
         eth_header_t *eth_header = (eth_header_t *) output_buffer;
-        memcpy(eth_header->dest_addr, auth_data->bssid.address, ETH_ADDR);
+        memcpy(eth_header->dest_addr, auth_data->bssid.address, ETH_ADDR);
         memcpy(eth_header->src_addr, nic_address.address, ETH_ADDR);
         eth_header->proto = host2uint16_t_be(ETH_TYPE_PAE);
-
+        
         ieee80211_eapol_key_frame_t *output_key_frame =
-                (ieee80211_eapol_key_frame_t *) 
-                (output_buffer + sizeof(eth_header_t));
-
+            (ieee80211_eapol_key_frame_t *)
+            (output_buffer + sizeof(eth_header_t));
+        
         /* Copy content of incoming EAPOL-Key frame. */
-        memcpy((void *) output_key_frame, buffer, 
-                sizeof(ieee80211_eapol_key_frame_t));
-
+        memcpy((void *) output_key_frame, buffer,
+            sizeof(ieee80211_eapol_key_frame_t));
+        
         output_key_frame->proto_version = 0x1;
         output_key_frame->body_length =
-                host2uint16_t_be(output_size - sizeof(eth_header_t) - 4);
-        output_key_frame->key_info &= 
-                ~host2uint16_t_be(
-                        IEEE80211_EAPOL_KEY_KEYINFO_ACK
-                );
-
-        if(key_phase) {
-                output_key_frame->key_info &= 
-                        ~host2uint16_t_be(
-                                IEEE80211_EAPOL_KEY_KEYINFO_ENCDATA
-                        );
-                output_key_frame->key_info &= 
-                        ~host2uint16_t_be(
-                                IEEE80211_EAPOL_KEY_KEYINFO_INSTALL
-                        );
+            host2uint16_t_be(output_size - sizeof(eth_header_t) - 4);
+        output_key_frame->key_info &=
+            ~host2uint16_t_be(IEEE80211_EAPOL_KEY_KEYINFO_ACK);
+        
+        if (key_phase) {
+                output_key_frame->key_info &=
+                    ~host2uint16_t_be(IEEE80211_EAPOL_KEY_KEYINFO_ENCDATA);
+                output_key_frame->key_info &=
+                    ~host2uint16_t_be(IEEE80211_EAPOL_KEY_KEYINFO_INSTALL);
                 output_key_frame->key_data_length = 0;
                 memset(output_key_frame->key_nonce, 0, 32);
@@ -1484 +1482 @@
                 memset(output_key_frame->key_rsc, 0, 8);
                 memset(output_key_frame->eapol_key_iv, 0, 16);
-
+                
                 /* Derive GTK and save it. */
-                if(final_phase) {
-                        uint16_t key_data_length = 
-                                uint16_t_be2host(key_frame->key_data_length);
+                if (final_phase) {
+                        uint16_t key_data_length =
+                            uint16_t_be2host(key_frame->key_data_length);
                         uint8_t key_data[key_data_length];
-                        uint8_t *data_ptr = (uint8_t *) (buffer + 
-                                sizeof(ieee80211_eapol_key_frame_t));
-
+                        uint8_t *data_ptr = (uint8_t *)
+                            (buffer + sizeof(ieee80211_eapol_key_frame_t));
+                        
                         int rc;
                         uint8_t work_key[32];
-                
-                        if(ccmp_used) {
-                                rc = ieee80211_aes_key_unwrap(ptk + KEK_OFFSET, 
-                                        data_ptr, key_data_length, key_data);
+                        
+                        if (ccmp_used) {
+                                rc = ieee80211_aes_key_unwrap(ptk + KEK_OFFSET,
+                                    data_ptr, key_data_length, key_data);
                         } else {
                                 memcpy(work_key, key_frame->eapol_key_iv, 16);
                                 memcpy(work_key + 16, ptk + KEK_OFFSET, 16);
-                                rc = ieee80211_rc4_key_unwrap(work_key, 
-                                        data_ptr, key_data_length, key_data);
+                                rc = ieee80211_rc4_key_unwrap(work_key,
+                                    data_ptr, key_data_length, key_data);
                         }
                         
-                        if(rc == EOK) {
+                        if (rc == EOK) {
                                 uint8_t *key_data_ptr = old_wpa ? key_data :
-                                        ieee80211_process_ies(ieee80211_dev,
-                                        NULL, key_data, key_data_length);
-
-                                if(key_data_ptr) {
+                                    ieee80211_process_ies(ieee80211_dev,
+                                    NULL, key_data, key_data_length);
+                                
+                                if (key_data_ptr) {
                                         uint8_t *key_ptr;
-                                        if(old_wpa) {
+                                        
+                                        if (old_wpa)
                                                 key_ptr = key_data_ptr;
-                                        } else {
+                                        else {
                                                 gtk_id = *key_data_ptr & 0x3;
                                                 key_ptr = key_data_ptr + 2;
                                         }
-                                                
+                                        
                                         memcpy(gtk, key_ptr, gtk_key_length);
                                         handshake_done = true;
@@ -1526 +1525 @@
                 }
         } else {
-                output_key_frame->key_info |= 
-                        host2uint16_t_be(
-                                IEEE80211_EAPOL_KEY_KEYINFO_MIC
-                        );
+                output_key_frame->key_info |=
+                    host2uint16_t_be(IEEE80211_EAPOL_KEY_KEYINFO_MIC);
                 output_key_frame->key_data_length =
-                        host2uint16_t_be(auth_link->auth_ie_len);
-                memcpy((void *)output_key_frame + 
-                        sizeof(ieee80211_eapol_key_frame_t),
-                        auth_link->auth_ie,
-                        auth_link->auth_ie_len);
-
+                    host2uint16_t_be(auth_link->auth_ie_len);
+                memcpy((void *) output_key_frame +
+                    sizeof(ieee80211_eapol_key_frame_t),
+                    auth_link->auth_ie, auth_link->auth_ie_len);
+                
                 /* Compute PMK. */
                 uint8_t pmk[PBKDF2_KEY_LENGTH];
                 pbkdf2((uint8_t *) ieee80211_dev->bssid_info.password,
-                        str_size(ieee80211_dev->bssid_info.password),
-                        (uint8_t *) auth_data->ssid,
-                        str_size(auth_data->ssid), pmk);
-
+                    str_size(ieee80211_dev->bssid_info.password),
+                    (uint8_t *) auth_data->ssid,
+                    str_size(auth_data->ssid), pmk);
+                
                 uint8_t *anonce = key_frame->key_nonce;
-
+                
                 /* Generate SNONCE. */
                 uint8_t snonce[32];
                 rnd_sequence(snonce, 32);
-
+                
                 memcpy(output_key_frame->key_nonce, snonce, 32);
-
+                
                 uint8_t *dest_addr = eth_header->dest_addr;
                 uint8_t *src_addr = eth_header->src_addr;
-
+                
                 /* Derive PTK and save it. */
                 uint8_t crypt_data[PRF_CRYPT_DATA_LENGTH];
-                memcpy(crypt_data, 
-                        min_sequence(dest_addr, src_addr, ETH_ADDR), 
-                        ETH_ADDR);
-                memcpy(crypt_data + ETH_ADDR, 
-                        max_sequence(dest_addr, src_addr, ETH_ADDR), 
-                        ETH_ADDR);
-                memcpy(crypt_data + 2*ETH_ADDR, 
-                        min_sequence(anonce, snonce, 32), 
-                        32);
-                memcpy(crypt_data + 2*ETH_ADDR + 32, 
-                        max_sequence(anonce, snonce, 32),
-                        32);
+                memcpy(crypt_data,
+                    min_sequence(dest_addr, src_addr, ETH_ADDR), ETH_ADDR);
+                memcpy(crypt_data + ETH_ADDR,
+                    max_sequence(dest_addr, src_addr, ETH_ADDR), ETH_ADDR);
+                memcpy(crypt_data + 2*ETH_ADDR,
+                    min_sequence(anonce, snonce, 32), 32);
+                memcpy(crypt_data + 2*ETH_ADDR + 32,
+                    max_sequence(anonce, snonce, 32), 32);
                 ieee80211_prf(pmk, crypt_data, ptk, ptk_key_length);
         }
-
+        
         /* Compute MIC of key frame data from KCK part of PTK. */
         uint8_t mic[mic_hash];
-        hmac(ptk, 16, (uint8_t *) output_key_frame, 
-                output_size - sizeof(eth_header_t), mic, mic_hash);
-
+        hmac(ptk, 16, (uint8_t *) output_key_frame,
+            output_size - sizeof(eth_header_t), mic, mic_hash);
+        
         memcpy(output_key_frame->key_mic, mic, 16);
-
+        
         ieee80211_send_frame(nic, output_buffer, output_size);
-
+        
         free(output_buffer);
-
+        
         ieee80211_key_config_t key_config;
         
         /* Insert Pairwise key. */
-        if((key_phase && old_wpa) || (final_phase && !old_wpa)) {
+        if ((key_phase && old_wpa) || (final_phase && !old_wpa)) {
                 key_config.suite = auth_data->security.pair_alg;
                 key_config.flags =
-                        IEEE80211_KEY_FLAG_TYPE_PAIRWISE;
-                memcpy(key_config.data, 
-                        ptk + TK_OFFSET,
-                        ptk_key_length - TK_OFFSET);
-
+                    IEEE80211_KEY_FLAG_TYPE_PAIRWISE;
+                memcpy(key_config.data,
+                    ptk + TK_OFFSET, ptk_key_length - TK_OFFSET);
+                
                 ieee80211_dev->ops->key_config(ieee80211_dev,
-                        &key_config, true);
+                    &key_config, true);
         }
         
         /* Insert Group key. */
-        if(final_phase) {
+        if (final_phase) {
                 key_config.id = gtk_id;
                 key_config.suite = auth_data->security.group_alg;
-                key_config.flags =
-                        IEEE80211_KEY_FLAG_TYPE_GROUP;
+                key_config.flags = IEEE80211_KEY_FLAG_TYPE_GROUP;
                 memcpy(key_config.data, gtk, gtk_key_length);
-
+                
                 ieee80211_dev->ops->key_config(ieee80211_dev,
-                        &key_config, true);
-        }
-
+                    &key_config, true);
+        }
+        
         /* Signal successful handshake completion. */
-        if(handshake_done) {
+        if (handshake_done) {
                 fibril_mutex_lock(&ieee80211_dev->gen_mutex);
                 fibril_condvar_signal(&ieee80211_dev->gen_cond);
@@ -1621 +1611 @@
 
 static int ieee80211_process_eapol_frame(ieee80211_dev_t *ieee80211_dev,
-        void *buffer, size_t buffer_size)
-{
-        ieee80211_eapol_key_frame_t *key_frame = 
-                (ieee80211_eapol_key_frame_t *) buffer;
-        if(ieee80211_is_eapol_key_frame(key_frame)) {
+    void *buffer, size_t buffer_size)
+{
+        ieee80211_eapol_key_frame_t *key_frame =
+            (ieee80211_eapol_key_frame_t *) buffer;
+        
+        if (ieee80211_is_eapol_key_frame(key_frame))
                 return ieee80211_process_4way_handshake(ieee80211_dev, buffer,
-                        buffer_size);
-        }
+                    buffer_size);
         
         return EOK;
 }
 
-/**
- * Process data frame.
- * 
+/** Process data frame.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param buffer Data buffer starting with IEEE 802.11 data header.
- * @param buffer_size Size of buffer.
- * 
+ * @param buffer        Data buffer starting with IEEE 802.11 data header.
+ * @param buffer_size   Size of buffer.
+ *
  * @return EOK if succeed, negative error code otherwise.
- */
-static int ieee80211_process_data(ieee80211_dev_t *ieee80211_dev, 
-        void *buffer, size_t buffer_size)
-{
-        ieee80211_data_header_t *data_header = 
-                (ieee80211_data_header_t *) buffer;
-        
-        if(ieee80211_has_data_frame(data_header->frame_ctrl)) {
+ *
+ */
+static int ieee80211_process_data(ieee80211_dev_t *ieee80211_dev,
+    void *buffer, size_t buffer_size)
+{
+        ieee80211_data_header_t *data_header =
+            (ieee80211_data_header_t *) buffer;
+        
+        if (ieee80211_has_data_frame(data_header->frame_ctrl)) {
                 nic_t *nic = nic_get_from_ddf_dev(ieee80211_dev->ddf_dev);
-                size_t strip_length = sizeof(ieee80211_data_header_t) + 
-                        ARRAY_SIZE(rfc1042_header);
+                size_t strip_length = sizeof(ieee80211_data_header_t) +
+                    ARRAY_SIZE(rfc1042_header);
                 
                 /* TODO: Different by used security alg. */
                 /* TODO: Trim frame by used security alg. */
                 // TODO: Distinguish used key (pair/group) by dest address ?
-                if(ieee80211_is_encrypted_frame(data_header->frame_ctrl)) {
+                if (ieee80211_is_encrypted_frame(data_header->frame_ctrl))
                         strip_length += 8;
-                }
                 
                 /* Process 4-way authentication handshake. */
                 uint16_t *proto = (uint16_t *) (buffer + strip_length);
-                if(uint16_t_be2host(*proto) == ETH_TYPE_PAE) {
+                if (uint16_t_be2host(*proto) == ETH_TYPE_PAE)
                         return ieee80211_process_eapol_frame(ieee80211_dev,
-                                buffer + strip_length + sizeof(uint16_t),
-                                buffer_size - strip_length - sizeof(uint16_t));
-                }
-                
-                /* Note: ETH protocol ID is already there, so we don't create 
-                 * whole ETH header. */
-                size_t frame_size = 
-                        buffer_size - strip_length + sizeof(eth_header_t)-2;
+                            buffer + strip_length + sizeof(uint16_t),
+                            buffer_size - strip_length - sizeof(uint16_t));
+                
+                /*
+                 * Note: ETH protocol ID is already there, so we don't create
+                 * whole ETH header.
+                 */
+                size_t frame_size =
+                    buffer_size - strip_length + sizeof(eth_header_t) - 2;
                 nic_frame_t *frame = nic_alloc_frame(nic, frame_size);
-
-                if(frame == NULL) {
+                
+                if(frame == NULL)
                         return ENOMEM;
-                }
-
-                uint8_t *src_addr = 
-                        ieee80211_is_fromds_frame(data_header->frame_ctrl) ?
-                                data_header->address3 : data_header->address2;
-                uint8_t *dest_addr = 
-                        ieee80211_is_tods_frame(data_header->frame_ctrl) ?
-                                data_header->address3 : data_header->address1;
-
-                eth_header_t *eth_header =
-                        (eth_header_t *) frame->data;
+                
+                uint8_t *src_addr =
+                    ieee80211_is_fromds_frame(data_header->frame_ctrl) ?
+                    data_header->address3 : data_header->address2;
+                uint8_t *dest_addr =
+                    ieee80211_is_tods_frame(data_header->frame_ctrl) ?
+                    data_header->address3 : data_header->address1;
+                
+                eth_header_t *eth_header = (eth_header_t *) frame->data;
                 memcpy(eth_header->src_addr, src_addr, ETH_ADDR);
                 memcpy(eth_header->dest_addr, dest_addr, ETH_ADDR);
-
-                memcpy(frame->data + sizeof(eth_header_t)-2, 
-                        buffer + strip_length, 
-                        buffer_size - strip_length);
-
+                
+                memcpy(frame->data + sizeof(eth_header_t) - 2,
+                    buffer + strip_length, buffer_size - strip_length);
+                
                 nic_received_frame(nic, frame);
         }
@@ -1700 +1687 @@
 }
 
-/**
- * IEEE 802.11 RX frames handler.
- * 
+/** IEEE 802.11 RX frames handler.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device structure.
- * @param buffer Buffer with data.
- * @param buffer_size Size of buffer.
- * 
+ * @param buffer        Buffer with data.
+ * @param buffer_size   Size of buffer.
+ *
  * @return EOK if succeed, negative error code otherwise.
+ *
  */
 int ieee80211_rx_handler(ieee80211_dev_t *ieee80211_dev, void *buffer,
-        size_t buffer_size)
+    size_t buffer_size)
 {
         uint16_t frame_ctrl = *((uint16_t *) buffer);
-        if(ieee80211_is_mgmt_frame(frame_ctrl)) {
+        
+        if (ieee80211_is_mgmt_frame(frame_ctrl)) {
                 ieee80211_mgmt_header_t *mgmt_header =
-                        (ieee80211_mgmt_header_t *) buffer;
-                
-                if(ieee80211_is_probe_response_frame(mgmt_header->frame_ctrl) ||
-                        ieee80211_is_beacon_frame(mgmt_header->frame_ctrl)) {
+                    (ieee80211_mgmt_header_t *) buffer;
+                
+                if ((ieee80211_is_probe_response_frame(mgmt_header->frame_ctrl)) ||
+                    (ieee80211_is_beacon_frame(mgmt_header->frame_ctrl)))
                         return ieee80211_process_probe_response(ieee80211_dev,
-                                mgmt_header, buffer_size);
-                }
-                
-                if(ieee80211_is_auth_frame(mgmt_header->frame_ctrl)) {
+                            mgmt_header, buffer_size);
+                
+                if (ieee80211_is_auth_frame(mgmt_header->frame_ctrl))
                         return ieee80211_process_auth_response(ieee80211_dev,
-                                mgmt_header);
-                }
-                
-                if(ieee80211_is_assoc_response_frame(mgmt_header->frame_ctrl)) {
+                            mgmt_header);
+                
+                if (ieee80211_is_assoc_response_frame(mgmt_header->frame_ctrl))
                         return ieee80211_process_assoc_response(ieee80211_dev,
-                                mgmt_header);
-                }
-        } else if(ieee80211_is_data_frame(frame_ctrl)) {
-                return ieee80211_process_data(ieee80211_dev, buffer, 
-                        buffer_size);
-        }
+                            mgmt_header);
+        } else if (ieee80211_is_data_frame(frame_ctrl))
+                return ieee80211_process_data(ieee80211_dev, buffer,
+                    buffer_size);
         
         return EOK;

uspace/lib/ieee80211/src/ieee80211_iface_impl.c

@@ -27 +27 @@
  */
 
+/** @addtogroup libieee80211
+ * @{
+ */
+
+/** @file ieee80211_iface_impl.c
+ *
+ * IEEE 802.11 default interface functions implementation.
+ */
+
 #include <str.h>
 #include <errno.h>
-
 #include <ieee80211_private.h>
 #include <ieee80211_iface_impl.h>
 
-/** @addtogroup libieee80211
- * @{
- */
-
-/** @file ieee80211_iface_impl.c
- * 
- * IEEE 802.11 default interface functions implementation.
- */
-
-/**
- * Implementation of fetching scan results.
- * 
- * @param fun Device function.
+/** Implementation of fetching scan results.
+ *
+ * @param fun     Device function.
  * @param results Structure where should be stored scan results.
- * 
- * @return EOK if everything went OK, EREFUSED when device is not ready yet.
- */
-int ieee80211_get_scan_results_impl(ddf_fun_t *fun, 
-        ieee80211_scan_results_t *results, bool now)
+ *
+ * @return EOK if everything went OK,
+ *         EREFUSED when device is not ready yet.
+ *
+ */
+int ieee80211_get_scan_results_impl(ddf_fun_t *fun,
+    ieee80211_scan_results_t *results, bool now)
 {
         nic_t *nic_data = nic_get_from_ddf_fun(fun);
         ieee80211_dev_t *ieee80211_dev = nic_get_specific(nic_data);
         
-        if(!ieee80211_is_ready(ieee80211_dev))
+        if (!ieee80211_is_ready(ieee80211_dev))
                 return EREFUSED;
         
-        if(now) {
+        if (now)
                 ieee80211_dev->ops->scan(ieee80211_dev);
-        }
         
         fibril_mutex_lock(&ieee80211_dev->ap_list.results_mutex);
-        if(results) {
+        
+        if (results) {
                 ieee80211_scan_result_list_t *result_list =
-                        &ieee80211_dev->ap_list;
-                
-                int i = 0;
+                    &ieee80211_dev->ap_list;
+                
+                unsigned int i = 0;
                 ieee80211_scan_result_list_foreach(*result_list, result) {
-                        memcpy(&results->results[i], 
-                                &result->scan_result, 
-                                sizeof(ieee80211_scan_result_t));
+                        memcpy(&results->results[i],
+                            &result->scan_result,
+                            sizeof(ieee80211_scan_result_t));
                         i++;
                 }
@@ -78 +78 @@
                 results->length = i;
         }
+        
         fibril_mutex_unlock(&ieee80211_dev->ap_list.results_mutex);
         
@@ -88 +89 @@
 }
 
-/**
- * Working procedure of connect function.
- * 
+/** Working procedure of connect function.
+ *
  * @param ieee80211_dev Pointer to IEEE 802.11 device.
- * @param auth_data Selected AP data we want to connect to.
- * 
- * @return EOK if everything OK, ETIMEOUT when timeout during authenticating.
+ * @param auth_data     Selected AP data we want to connect to.
+ *
+ * @return EOK if everything OK,
+ *         ETIMEOUT when timeout during authenticating.
+ *
  */
 static int ieee80211_connect_proc(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_scan_result_link_t *auth_data, char *password)
+    ieee80211_scan_result_link_t *auth_data, char *password)
 {
         ieee80211_dev->bssid_info.res_link = auth_data;
         
         /* Set channel. */
-        int rc = ieee80211_dev->ops->set_freq(ieee80211_dev, 
-                ieee80211_channel_to_freq(auth_data->scan_result.channel));
-        if(rc != EOK)
+        int rc = ieee80211_dev->ops->set_freq(ieee80211_dev,
+            ieee80211_channel_to_freq(auth_data->scan_result.channel));
+        if (rc != EOK)
                 return rc;
         
         /* Try to authenticate. */
         ieee80211_authenticate(ieee80211_dev);
+        
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
         rc = fibril_condvar_wait_timeout(&ieee80211_dev->gen_cond,
-                        &ieee80211_dev->gen_mutex,
-                        AUTH_TIMEOUT);
+            &ieee80211_dev->gen_mutex, AUTH_TIMEOUT);
         fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
-        if(rc != EOK)
+        
+        if (rc != EOK)
                 return rc;
-        if(ieee80211_get_auth_phase(ieee80211_dev) != 
-                IEEE80211_AUTH_AUTHENTICATED) {
-                ieee80211_set_auth_phase(ieee80211_dev, 
-                        IEEE80211_AUTH_DISCONNECTED);
+        
+        if (ieee80211_get_auth_phase(ieee80211_dev) !=
+            IEEE80211_AUTH_AUTHENTICATED) {
+                ieee80211_set_auth_phase(ieee80211_dev,
+                    IEEE80211_AUTH_DISCONNECTED);
                 return EINVAL;
         }
@@ -125 +129 @@
         /* Try to associate. */
         ieee80211_associate(ieee80211_dev, password);
+        
         fibril_mutex_lock(&ieee80211_dev->gen_mutex);
         rc = fibril_condvar_wait_timeout(&ieee80211_dev->gen_cond,
-                        &ieee80211_dev->gen_mutex,
-                        AUTH_TIMEOUT);
+            &ieee80211_dev->gen_mutex, AUTH_TIMEOUT);
         fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
-        if(rc != EOK)
+        
+        if (rc != EOK)
                 return rc;
-        if(ieee80211_get_auth_phase(ieee80211_dev) != 
-                IEEE80211_AUTH_ASSOCIATED) {
-                ieee80211_set_auth_phase(ieee80211_dev, 
-                        IEEE80211_AUTH_DISCONNECTED);
+        
+        if (ieee80211_get_auth_phase(ieee80211_dev) !=
+            IEEE80211_AUTH_ASSOCIATED) {
+                ieee80211_set_auth_phase(ieee80211_dev,
+                    IEEE80211_AUTH_DISCONNECTED);
                 return EINVAL;
         }
         
         /* On open network, we are finished. */
-        if(auth_data->scan_result.security.type != IEEE80211_SECURITY_OPEN) {
+        if (auth_data->scan_result.security.type !=
+            IEEE80211_SECURITY_OPEN) {
                 /* Otherwise wait for 4-way handshake to complete. */
+                
                 fibril_mutex_lock(&ieee80211_dev->gen_mutex);
                 rc = fibril_condvar_wait_timeout(&ieee80211_dev->gen_cond,
-                                &ieee80211_dev->gen_mutex,
-                                HANDSHAKE_TIMEOUT);
+                    &ieee80211_dev->gen_mutex, HANDSHAKE_TIMEOUT);
                 fibril_mutex_unlock(&ieee80211_dev->gen_mutex);
-                if(rc != EOK) {
+                
+                if (rc != EOK) {
                         ieee80211_deauthenticate(ieee80211_dev);
                         return rc;
@@ -158 +166 @@
 }
 
-/**
- * Implementation of connecting to specified SSID.
- * 
- * @param fun Device function.
+/** Implementation of connecting to specified SSID.
+ *
+ * @param fun        Device function.
  * @param ssid_start SSID prefix of access point we want to connect to.
- * 
- * @return EOK if everything OK, ETIMEOUT when timeout during authenticating,
- * EINVAL when SSID not in scan results list, EPERM when incorrect password
- * passed, EREFUSED when device is not ready yet.
+ *
+ * @return EOK if everything OK,
+ *         ETIMEOUT when timeout during authenticating,
+ *         EINVAL when SSID not in scan results list,
+ *         EPERM when incorrect password passed,
+ *         EREFUSED when device is not ready yet.
+ *
  */
 int ieee80211_connect_impl(ddf_fun_t *fun, char *ssid_start, char *password)
@@ -173 +183 @@
         assert(password);
         
-        int rc;
-        
         nic_t *nic_data = nic_get_from_ddf_fun(fun);
         ieee80211_dev_t *ieee80211_dev = nic_get_specific(nic_data);
         
-        if(!ieee80211_is_ready(ieee80211_dev))
+        if (!ieee80211_is_ready(ieee80211_dev))
                 return EREFUSED;
         
-        if(ieee80211_is_connected(ieee80211_dev)) {
-                rc = ieee80211_dev->iface->disconnect(fun);
-                if(rc != EOK)
+        if (ieee80211_is_connected(ieee80211_dev)) {
+                int rc = ieee80211_dev->iface->disconnect(fun);
+                if (rc != EOK)
                         return rc;
         }
@@ -189 +197 @@
         ieee80211_set_connect_request(ieee80211_dev);
         
-        rc = ENOENT;
+        int rc = ENOENT;
         fibril_mutex_lock(&ieee80211_dev->scan_mutex);
         
         ieee80211_dev->pending_conn_req = false;
-
+        
         ieee80211_scan_result_list_foreach(ieee80211_dev->ap_list, result) {
-                if(!str_lcmp(ssid_start, 
-                        result->scan_result.ssid, 
-                        str_size(ssid_start))) {
+                if (!str_lcmp(ssid_start, result->scan_result.ssid,
+                    str_size(ssid_start))) {
                         rc = ieee80211_connect_proc(ieee80211_dev, result,
-                                password);
+                            password);
                         break;
                 }
@@ -209 +216 @@
 }
 
-/**
- * Implementation of disconnecting device from network.
- * 
+/** Implementation of disconnecting device from network.
+ *
  * @param fun Device function.
- * 
- * @return EOK if everything OK, EREFUSED if device is not ready yet.
+ *
+ * @return EOK if everything OK,
+ *         EREFUSED if device is not ready yet.
+ *
  */
 int ieee80211_disconnect_impl(ddf_fun_t *fun)
@@ -221 +229 @@
         ieee80211_dev_t *ieee80211_dev = nic_get_specific(nic_data);
         
-        if(!ieee80211_is_ready(ieee80211_dev))
+        if (!ieee80211_is_ready(ieee80211_dev))
                 return EREFUSED;
         
-        if(!ieee80211_is_connected(ieee80211_dev)) {
+        if (!ieee80211_is_connected(ieee80211_dev))
                 return EOK;
-        } else {
-                fibril_mutex_lock(&ieee80211_dev->ap_list.results_mutex);
-                int rc = ieee80211_deauthenticate(ieee80211_dev);
-                fibril_mutex_unlock(&ieee80211_dev->ap_list.results_mutex);
-                return rc;
-        }
+        
+        fibril_mutex_lock(&ieee80211_dev->ap_list.results_mutex);
+        int rc = ieee80211_deauthenticate(ieee80211_dev);
+        fibril_mutex_unlock(&ieee80211_dev->ap_list.results_mutex);
+        
+        return rc;
 }
 

uspace/lib/ieee80211/src/ieee80211_impl.c

@@ -32 +32 @@
 
 /** @file ieee80211_impl.c
- * 
+ *
  * IEEE 802.11 default device functions implementation.
  */
@@ -40 +40 @@
 #include <stdlib.h>
 #include <errno.h>
-
 #include <ieee80211_impl.h>
 
-/**
- * Default implementation of IEEE802.11 start function.
- * 
- * @param ieee80211_dev Structure of IEEE802.11 device.
- * 
- * @return EOK. 
+/** Default implementation of IEEE802.11 start function.
+ *
+ * @param ieee80211_dev Structure of IEEE802.11 device.
+ *
+ * @return EOK.
+ *
  */
 int ieee80211_start_impl(ieee80211_dev_t *ieee80211_dev)
@@ -55 +54 @@
 }
 
-/**
- * Default implementation of IEEE802.11 TX handler function.
- * 
- * @param ieee80211_dev Structure of IEEE802.11 device.
- * @param buffer Buffer with data to send.
- * @param buffer_size Size of buffer.
- * 
- * @return EOK. 
- */
-int ieee80211_tx_handler_impl(ieee80211_dev_t *ieee80211_dev, void *buffer, 
-        size_t buffer_size)
-{
-        return EOK;
-}
-
-/**
- * Default implementation of IEEE802.11 set frequency function.
- * 
- * @param ieee80211_dev Structure of IEEE802.11 device.
- * @param freq Value of frequency to be switched on.
- * 
- * @return EOK. 
+/** Default implementation of IEEE802.11 TX handler function.
+ *
+ * @param ieee80211_dev Structure of IEEE802.11 device.
+ * @param buffer        Buffer with data to send.
+ * @param buffer_size   Size of buffer.
+ *
+ * @return EOK.
+ *
+ */
+int ieee80211_tx_handler_impl(ieee80211_dev_t *ieee80211_dev, void *buffer,
+    size_t buffer_size)
+{
+        return EOK;
+}
+
+/** Default implementation of IEEE802.11 set frequency function.
+ *
+ * @param ieee80211_dev Structure of IEEE802.11 device.
+ * @param freq          Value of frequency to be switched on.
+ *
+ * @return EOK.
+ *
  */
 int ieee80211_set_freq_impl(ieee80211_dev_t *ieee80211_dev, uint16_t freq)
@@ -83 +82 @@
 }
 
-/**
- * Default implementation of IEEE802.11 BSSID change function.
- * 
- * @param ieee80211_dev Structure of IEEE802.11 device.
- * 
- * @return EOK. 
- */
-int ieee80211_bssid_change_impl(ieee80211_dev_t *ieee80211_dev, 
-        bool connected)
-{
-        return EOK;
-}
-
-/**
- * Default implementation of IEEE802.11 key config function.
- * 
- * @param ieee80211_dev Structure of IEEE802.11 device.
- * 
- * @return EOK. 
+/** Default implementation of IEEE802.11 BSSID change function.
+ *
+ * @param ieee80211_dev Structure of IEEE802.11 device.
+ *
+ * @return EOK.
+ *
+ */
+int ieee80211_bssid_change_impl(ieee80211_dev_t *ieee80211_dev,
+    bool connected)
+{
+        return EOK;
+}
+
+/** Default implementation of IEEE802.11 key config function.
+ *
+ * @param ieee80211_dev Structure of IEEE802.11 device.
+ *
+ * @return EOK.
+ *
  */
 int ieee80211_key_config_impl(ieee80211_dev_t *ieee80211_dev,
-        ieee80211_key_config_t *key_conf, bool insert)
-{
-        return EOK;
-}
-
-/**
- * Default implementation of IEEE802.11 scan function.
- * 
- * @param ieee80211_dev Structure of IEEE802.11 device.
- * @param clear Whether to clear current scan results.
- * 
- * @return EOK if succeed, negative error code otherwise. 
+    ieee80211_key_config_t *key_conf, bool insert)
+{
+        return EOK;
+}
+
+/** Default implementation of IEEE802.11 scan function.
+ *
+ * @param ieee80211_dev Structure of IEEE802.11 device.
+ * @param clear         Whether to clear current scan results.
+ *
+ * @return EOK if succeed, negative error code otherwise.
+ *
  */
 int ieee80211_scan_impl(ieee80211_dev_t *ieee80211_dev)
@@ -121 +120 @@
         fibril_mutex_lock(&ieee80211_dev->scan_mutex);
         
-        if(ieee80211_get_auth_phase(ieee80211_dev) == 
-                IEEE80211_AUTH_DISCONNECTED) {
+        if (ieee80211_get_auth_phase(ieee80211_dev) ==
+            IEEE80211_AUTH_DISCONNECTED) {
                 fibril_mutex_lock(&ieee80211_dev->ap_list.results_mutex);
+                
                 /* Remove old entries we don't receive beacons from. */
-                ieee80211_scan_result_list_t *result_list = 
-                        &ieee80211_dev->ap_list;
+                ieee80211_scan_result_list_t *result_list =
+                    &ieee80211_dev->ap_list;
+                
                 list_foreach_safe(result_list->list, cur_link, next_link) {
-                        ieee80211_scan_result_link_t *cur_result = 
-                                list_get_instance(cur_link,
-                                ieee80211_scan_result_link_t, 
-                                link);
-                        if((time(NULL) - cur_result->last_beacon) > 
-                                MAX_KEEP_SCAN_SPAN_SEC) {
-                                ieee80211_scan_result_list_remove(result_list, 
-                                        cur_result);
-                        }
+                        ieee80211_scan_result_link_t *cur_result =
+                            list_get_instance(cur_link,
+                            ieee80211_scan_result_link_t, link);
+                        
+                        if ((time(NULL) - cur_result->last_beacon) >
+                            MAX_KEEP_SCAN_SPAN_SEC)
+                                ieee80211_scan_result_list_remove(result_list,
+                                    cur_result);
                 }
+                
                 fibril_mutex_unlock(&ieee80211_dev->ap_list.results_mutex);
-
+                
                 uint16_t orig_freq = ieee80211_dev->current_freq;
-
-                for(uint16_t freq = IEEE80211_FIRST_FREQ;
-                        freq <= IEEE80211_MAX_FREQ; 
-                        freq += IEEE80211_CHANNEL_GAP) {
-                        if(ieee80211_pending_connect_request(ieee80211_dev)) {
+                
+                for (uint16_t freq = IEEE80211_FIRST_FREQ;
+                    freq <= IEEE80211_MAX_FREQ; freq += IEEE80211_CHANNEL_GAP) {
+                        if (ieee80211_pending_connect_request(ieee80211_dev))
                                 break;
-                        }
                         
                         ieee80211_dev->ops->set_freq(ieee80211_dev, freq);
                         ieee80211_probe_request(ieee80211_dev, NULL);
-
+                        
                         /* Wait for probe responses. */
                         async_usleep(SCAN_CHANNEL_WAIT_USEC);
                 }
-
+                
                 ieee80211_dev->ops->set_freq(ieee80211_dev, orig_freq);
         }
@@ -164 +163 @@
 }
 
-/**
- * Pseudorandom function used for IEEE 802.11 pairwise key computation
- * using SHA1 hash algorithm.
- * 
- * @param key Key with PBKDF2 encrypted passphrase.
- * @param data Concatenated sequence of both mac addresses and nonces.
- * @param hash Output parameter for result hash.
+/** Pseudorandom function used for IEEE 802.11 pairwise key computation.
+ *
+ * Using SHA1 hash algorithm.
+ *
+ * @param key         Key with PBKDF2 encrypted passphrase.
+ * @param data        Concatenated sequence of both MAC
+ *                    addresses and nonces.
+ * @param hash        Output parameter for result hash.
  * @param output_size Length of output sequence to be generated.
- * 
- * @return EINVAL when key or data not specified, ENOMEM when pointer for 
- * output hash result is not allocated, otherwise EOK. 
- */
-int ieee80211_prf(uint8_t *key, uint8_t *data, uint8_t *hash, 
-        size_t output_size)
-{
-        if(!key || !data)
+ *
+ * @return EINVAL when key or data not specified,
+ *         ENOMEM when pointer for output hash result
+ *         is not allocated, otherwise EOK.
+ *
+ */
+int ieee80211_prf(uint8_t *key, uint8_t *data, uint8_t *hash,
+    size_t output_size)
+{
+        if ((!key) || (!data))
                 return EINVAL;
         
-        if(!hash)
+        if (!hash)
                 return ENOMEM;
         
@@ -188 +190 @@
         
         const char *a = "Pairwise key expansion";
-        uint8_t result[HASH_SHA1*iters];
+        uint8_t result[HASH_SHA1 * iters];
         uint8_t temp[HASH_SHA1];
+        
         size_t data_size = PRF_CRYPT_DATA_LENGTH + str_size(a) + 2;
         uint8_t work_arr[data_size];
@@ -196 +199 @@
         memcpy(work_arr, a, str_size(a));
         memcpy(work_arr + str_size(a) + 1, data, PRF_CRYPT_DATA_LENGTH);
-
-        for(uint8_t i = 0; i < iters; i++) {
+        
+        for (uint8_t i = 0; i < iters; i++) {
                 memcpy(work_arr + data_size - 1, &i, 1);
-                hmac(key, PBKDF2_KEY_LENGTH, work_arr, data_size, temp, 
-                        HASH_SHA1);
+                hmac(key, PBKDF2_KEY_LENGTH, work_arr, data_size, temp,
+                    HASH_SHA1);
                 memcpy(result + i*HASH_SHA1, temp, HASH_SHA1);
         }
@@ -210 +213 @@
 
 int ieee80211_rc4_key_unwrap(uint8_t *key, uint8_t *data, size_t data_size,
-        uint8_t *output)
+    uint8_t *output)
 {
         return rc4(key, 32, data, data_size, 256, output);
@@ -216 +219 @@
 
 int ieee80211_aes_key_unwrap(uint8_t *kek, uint8_t *data, size_t data_size,
-        uint8_t *output)
-{
-        if(!kek || !data)
+    uint8_t *output)
+{
+        if ((!kek) || (!data))
                 return EINVAL;
         
-        if(!output)
+        if (!output)
                 return ENOMEM;
-
-        uint32_t n = data_size/8 - 1;
-        uint8_t work_data[n*8];
+        
+        uint32_t n = data_size / 8 - 1;
+        uint8_t work_data[n * 8];
         uint8_t work_input[AES_CIPHER_LENGTH];
         uint8_t work_output[AES_CIPHER_LENGTH];
         uint8_t *work_block;
         uint8_t a[8];
+        
         memcpy(a, data, 8);
-        uint64_t mask = 0xFF;
+        
+        uint64_t mask = 0xff;
         uint8_t shift, shb;
         
-        memcpy(work_data, data + 8, n*8);
-        for(int j = 5; j >= 0; j--) {
-                for(int i = n; i > 0; i--) {
-                        for(size_t k = 0; k < 8; k++) {
-                                shift = 56 - 8*k;
-                                shb = ((n*j+i) & (mask << shift)) >> shift;
+        memcpy(work_data, data + 8, n * 8);
+        for (int j = 5; j >= 0; j--) {
+                for (int i = n; i > 0; i--) {
+                        for (size_t k = 0; k < 8; k++) {
+                                shift = 56 - 8 * k;
+                                shb = ((n * j + i) & (mask << shift)) >> shift;
                                 a[k] ^= shb;
                         }
-                        work_block = work_data + (i-1)*8;
+                        
+                        work_block = work_data + (i - 1) * 8;
                         memcpy(work_input, a, 8);
                         memcpy(work_input + 8, work_block, 8);
                         aes_decrypt(kek, work_input, work_output);
                         memcpy(a, work_output, 8);
-                        memcpy(work_data + (i-1)*8, work_output + 8, 8);
+                        memcpy(work_data + (i - 1) * 8, work_output + 8, 8);
                 }
         }
         
         size_t it;
-        for(it = 0; it < 8; it++) {
-                if(a[it] != 0xA6)
+        for (it = 0; it < 8; it++) {
+                if (a[it] != 0xa6)
                         break;
         }
         
-        if(it == 8) {
-                memcpy(output, work_data, n*8);
+        if (it == 8) {
+                memcpy(output, work_data, n * 8);
                 return EOK;
-        } else {
-                return EINVAL;
-        }
-}
-
-static void ieee80211_michael_mic_block(uint32_t *l, uint32_t *r, 
-        uint32_t value)
+        }
+        
+        return EINVAL;
+}
+
+static void ieee80211_michael_mic_block(uint32_t *l, uint32_t *r,
+    uint32_t value)
 {
         *l ^= value;
         *r ^= rotl_uint32(*l, 17);
         *l += *r;
-        *r ^= ((*l & 0x00FF00FF) << 8) | ((*l & 0xFF00FF00) >> 8);
+        *r ^= ((*l & 0x00ff00ff) << 8) | ((*l & 0xff00ff00) >> 8);
         *l += *r;
         *r ^= rotl_uint32(*l, 3);
@@ -279 +285 @@
 }
 
-int ieee80211_michael_mic(uint8_t *key, uint8_t *buffer, size_t size, 
-        uint8_t *mic)
-{
-        if(!key || !buffer)
+int ieee80211_michael_mic(uint8_t *key, uint8_t *buffer, size_t size,
+    uint8_t *mic)
+{
+        if ((!key) || (!buffer))
                 return EINVAL;
         
-        if(!mic)
+        if (!mic)
                 return ENOMEM;
         
@@ -292 +298 @@
         
         ieee80211_data_header_t *data_header =
-                (ieee80211_data_header_t *) buffer;
-        
-        uint8_t *data = buffer + sizeof(ieee80211_data_header_t) + 
-                IEEE80211_TKIP_HEADER_LENGTH;
+            (ieee80211_data_header_t *) buffer;
+        
+        uint8_t *data = buffer + sizeof(ieee80211_data_header_t) +
+            IEEE80211_TKIP_HEADER_LENGTH;
         size_t data_size = size - sizeof(ieee80211_data_header_t) -
-                IEEE80211_TKIP_HEADER_LENGTH;
+            IEEE80211_TKIP_HEADER_LENGTH;
         
         /* Process header. */
-        uint8_t *src_addr = 
-                ieee80211_is_fromds_frame(data_header->frame_ctrl) ?
-                        data_header->address3 : data_header->address2;
-        uint8_t *dest_addr = 
-                ieee80211_is_tods_frame(data_header->frame_ctrl) ?
-                        data_header->address3 : data_header->address1;
+        uint8_t *src_addr =
+            ieee80211_is_fromds_frame(data_header->frame_ctrl) ?
+            data_header->address3 : data_header->address2;
+        uint8_t *dest_addr =
+            ieee80211_is_tods_frame(data_header->frame_ctrl) ?
+            data_header->address3 : data_header->address1;
         
         ieee80211_michael_mic_block(&l, &r, uint32le_from_seq(dest_addr));
-        ieee80211_michael_mic_block(&l, &r, 
-                uint16le_from_seq(dest_addr + 4) |
-                (uint16le_from_seq(src_addr) << 16));
+        ieee80211_michael_mic_block(&l, &r,
+            uint16le_from_seq(dest_addr + 4) |
+            (uint16le_from_seq(src_addr) << 16));
         ieee80211_michael_mic_block(&l, &r, uint32le_from_seq(src_addr + 2));
         ieee80211_michael_mic_block(&l, &r, 0);
@@ -318 +324 @@
         size_t pad = data_size % 4;
         
-        for(size_t k = 0; k < blocks; k++) {
-                ieee80211_michael_mic_block(&l, &r, 
-                        uint32le_from_seq(&data[k*4]));
+        for (size_t k = 0; k < blocks; k++) {
+                ieee80211_michael_mic_block(&l, &r,
+                    uint32le_from_seq(&data[k * 4]));
         }
         
         /* Add padding. */
-        uint32_t value = 0x5A;
-        for(size_t i = pad; i > 0; i--) {
+        uint32_t value = 0x5a;
+        for (size_t i = pad; i > 0; i--) {
                 value <<= 8;
-                value |= data[blocks*4 + (i-1)];
+                value |= data[blocks * 4 + (i - 1)];
         }
         
@@ -345 +351 @@
 {
         uint16_t *u16 = (uint16_t *) seq;
-        return uint16_t_le2host(*u16); 
+        return uint16_t_le2host(*u16);
 }
 
@@ -351 +357 @@
 {
         uint32_t *u32 = (uint32_t *) seq;
-        return uint32_t_le2host(*u32); 
+        return uint32_t_le2host(*u32);
 }
 
@@ -357 +363 @@
 {
         uint16_t *u16 = (uint16_t *) seq;
-        return uint16_t_be2host(*u16); 
+        return uint16_t_be2host(*u16);
 }
 
@@ -363 +369 @@
 {
         uint32_t *u32 = (uint32_t *) seq;
-        return uint32_t_be2host(*u32); 
+        return uint32_t_be2host(*u32);
 }
 
 int rnd_sequence(uint8_t *sequence, size_t length)
 {
-        if(!sequence)
+        if (!sequence)
                 return ENOMEM;
         
-        for(size_t i = 0; i < length; i++) {
+        for (size_t i = 0; i < length; i++)
                 sequence[i] = (uint8_t) rand();
-        }
         
         return EOK;
@@ -380 +385 @@
 uint8_t *min_sequence(uint8_t *seq1, uint8_t *seq2, size_t size)
 {
-        if(!seq1 || !seq2)
+        if ((!seq1) || (!seq2))
                 return NULL;
         
-        for(size_t i = 0; i < size; i++) {
-                if(seq1[i] < seq2[i]) {
+        for (size_t i = 0; i < size; i++) {
+                if (seq1[i] < seq2[i])
                         return seq1;
-                } else if(seq1[i] > seq2[i]) {
+                else if (seq1[i] > seq2[i])
                         return seq2;
-                }
         }
         
@@ -397 +401 @@
 {
         uint8_t *min = min_sequence(seq1, seq2, size);
-        if(min == seq1) {
+        if (min == seq1)
                 return seq2;
-        } else {
-                return seq1;
-        }
+        
+        return seq1;
 }