compile cleanup

4 files changed, 3 insertions, 4 deletions

diff --git a/src/firejail/fs_dev.c b/src/firejail/fs_dev.c
index 15eda6ea1..1039b151b 100644
--- a/src/firejail/fs_dev.c
+++ b/src/firejail/fs_dev.c
@@ -319,7 +319,7 @@ static void disable_file_or_dir(const char *fname) {
 }
 
 void fs_dev_disable_sound(void) {
-        int i = 0;
+        unsigned i = 0;
         while (dev[i].dev_fname != NULL) {
                 if (dev[i].type == DEV_SOUND)
                         disable_file_or_dir(dev[i].dev_fname);
diff --git a/src/fsec-optimize/main.c b/src/fsec-optimize/main.c
index d78d80c67..f4300f350 100644
--- a/src/fsec-optimize/main.c
+++ b/src/fsec-optimize/main.c
@@ -78,7 +78,7 @@ printf("\n");
                 return 1;
         }
         size = write(fd, outfilter, entries * sizeof(struct sock_filter));
-        if (size != entries * sizeof(struct sock_filter)) {
+        if (size != (int) (entries * sizeof(struct sock_filter))) {
                 fprintf(stderr, "Error: cannot write output file\n");
                 return 1;
         }
diff --git a/src/fsec-optimize/optimizer.c b/src/fsec-optimize/optimizer.c
index f5bef33e2..98e4aaa34 100644
--- a/src/fsec-optimize/optimizer.c
+++ b/src/fsec-optimize/optimizer.c
@@ -91,7 +91,7 @@ static int optimize_blacklists(struct sock_filter *filter, int entries) {
 
         // step 3: add the new ret KILL, and recalculate entries
         filter_step2[j].code = BPF_RET + BPF_K;
-        filter_step2[j].k == SECCOMP_RET_KILL;
+        filter_step2[j].k = SECCOMP_RET_KILL;
         entries = j + 1;
 
         // step 4: recalculate jumps
diff --git a/src/fsec-print/print.c b/src/fsec-print/print.c
index 794172a8f..e3b53c44c 100644
--- a/src/fsec-print/print.c
+++ b/src/fsec-print/print.c
@@ -305,7 +305,6 @@ static void bpf_decode_args(const struct sock_filter *bpf, unsigned int line) {
 }
 
 void print(struct sock_filter *filter, int entries) {
-        unsigned int line = 0;
         int i;
 
         /* header */