Skip to content

Commit

Permalink
warning fixes by crondaemon
Browse files Browse the repository at this point in the history
  • Loading branch information
vanhauser-thc committed Aug 18, 2017
1 parent b9465a4 commit fdf9b5f
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
1 change: 0 additions & 1 deletion Makefile.unix
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
CC=gcc
STRIP=strip
2 changes: 1 addition & 1 deletion hydra-mod.c
Original file line number Diff line number Diff line change
Expand Up @@ -954,7 +954,7 @@ char *hydra_receive_line(int32_t socket) {
if (got < 0) {
if (debug) {
sprintf(text, "[DEBUG] RECV [pid:%d]", getpid());
hydra_dump_data("", -1, text);
hydra_dump_data((unsigned char*)"", -1, text);
//hydra_report_debug(stderr, "DEBUG_RECV_BEGIN||END [pid:%d %d]", getpid(), i);
perror("recv");
}
Expand Down
2 changes: 1 addition & 1 deletion hydra-smb.c
Original file line number Diff line number Diff line change
Expand Up @@ -1499,7 +1499,7 @@ int32_t service_smb_init(char *ip, int32_t sp, unsigned char options, char *misc
return -1;
}

if (buf[15] & 16 == 16) {
if ((buf[15] & 16) == 16) {
fprintf(stderr, "[ERROR] target smb://%s:%d/ requires signing which we do not support\n", hostname, port);
return -1;
}
Expand Down

2 comments on commit fdf9b5f

@crondaemon
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why didn't you just merge my commit? Was the PR too large?

@vanhauser-thc
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

read my comment on your push request. you made some changes that I have bad experiences with :)

Please sign in to comment.