Mercurial > dropbear
comparison cli-kex.c @ 1094:c45d65392c1a
Fix pointer differ in signess warnings [-Werror=pointer-sign]
author | Gaël PORTAY <gael.portay@gmail.com> |
---|---|
date | Sat, 02 May 2015 15:59:06 +0200 |
parents | cbc73a5aefb0 |
children | aaf576b27a10 d7b752525b91 |
comparison
equal
deleted
inserted
replaced
1093:aae71c5f7d5b | 1094:c45d65392c1a |
---|---|
320 TRACE(("line is too short to be sensible")) | 320 TRACE(("line is too short to be sensible")) |
321 continue; | 321 continue; |
322 } | 322 } |
323 | 323 |
324 /* Compare hostnames */ | 324 /* Compare hostnames */ |
325 if (strncmp(cli_opts.remotehost, buf_getptr(line, hostlen), | 325 if (strncmp(cli_opts.remotehost, (const char *) buf_getptr(line, hostlen), |
326 hostlen) != 0) { | 326 hostlen) != 0) { |
327 continue; | 327 continue; |
328 } | 328 } |
329 | 329 |
330 buf_incrpos(line, hostlen); | 330 buf_incrpos(line, hostlen); |
332 /* there wasn't a space after the hostname, something dodgy */ | 332 /* there wasn't a space after the hostname, something dodgy */ |
333 TRACE(("missing space afte matching hostname")) | 333 TRACE(("missing space afte matching hostname")) |
334 continue; | 334 continue; |
335 } | 335 } |
336 | 336 |
337 if (strncmp(buf_getptr(line, algolen), algoname, algolen) != 0) { | 337 if (strncmp((const char *) buf_getptr(line, algolen), algoname, algolen) != 0) { |
338 TRACE(("algo doesn't match")) | 338 TRACE(("algo doesn't match")) |
339 continue; | 339 continue; |
340 } | 340 } |
341 | 341 |
342 buf_incrpos(line, algolen); | 342 buf_incrpos(line, algolen); |
344 TRACE(("missing space after algo")) | 344 TRACE(("missing space after algo")) |
345 continue; | 345 continue; |
346 } | 346 } |
347 | 347 |
348 /* Now we're at the interesting hostkey */ | 348 /* Now we're at the interesting hostkey */ |
349 ret = cmp_base64_key(keyblob, keybloblen, algoname, algolen, | 349 ret = cmp_base64_key(keyblob, keybloblen, (const unsigned char *) algoname, algolen, |
350 line, &fingerprint); | 350 line, &fingerprint); |
351 | 351 |
352 if (ret == DROPBEAR_SUCCESS) { | 352 if (ret == DROPBEAR_SUCCESS) { |
353 /* Good matching key */ | 353 /* Good matching key */ |
354 TRACE(("good matching key")) | 354 TRACE(("good matching key")) |
380 if (!cli_opts.always_accept_key) { | 380 if (!cli_opts.always_accept_key) { |
381 /* put the new entry in the file */ | 381 /* put the new entry in the file */ |
382 fseek(hostsfile, 0, SEEK_END); /* In case it wasn't opened append */ | 382 fseek(hostsfile, 0, SEEK_END); /* In case it wasn't opened append */ |
383 buf_setpos(line, 0); | 383 buf_setpos(line, 0); |
384 buf_setlen(line, 0); | 384 buf_setlen(line, 0); |
385 buf_putbytes(line, cli_opts.remotehost, hostlen); | 385 buf_putbytes(line, (const unsigned char *) cli_opts.remotehost, hostlen); |
386 buf_putbyte(line, ' '); | 386 buf_putbyte(line, ' '); |
387 buf_putbytes(line, algoname, algolen); | 387 buf_putbytes(line, (const unsigned char *) algoname, algolen); |
388 buf_putbyte(line, ' '); | 388 buf_putbyte(line, ' '); |
389 len = line->size - line->pos; | 389 len = line->size - line->pos; |
390 /* The only failure with base64 is buffer_overflow, but buf_getwriteptr | 390 /* The only failure with base64 is buffer_overflow, but buf_getwriteptr |
391 * will die horribly in the case anyway */ | 391 * will die horribly in the case anyway */ |
392 base64_encode(keyblob, keybloblen, buf_getwriteptr(line, len), &len); | 392 base64_encode(keyblob, keybloblen, buf_getwriteptr(line, len), &len); |