mirror of
https://github.com/opnsense/src.git
synced 2026-05-28 04:12:45 -04:00
Initialize variables and use byteorder(9) instead of aliasing char array
buf via uint32_t pointer. CID: 1375949 Reported by: Coverity, cem Reviewed by: cem MFC after: 3 weeks Sponsored by: iXsystems, Inc. Differential Revision: https://reviews.freebsd.org/D11153
This commit is contained in:
parent
c74ae2ca93
commit
426729f77f
1 changed files with 4 additions and 3 deletions
|
|
@ -32,6 +32,7 @@ __FBSDID("$FreeBSD$");
|
|||
#ifndef WITHOUT_CAPSICUM
|
||||
#include <sys/capsicum.h>
|
||||
#endif
|
||||
#include <sys/endian.h>
|
||||
#include <sys/socket.h>
|
||||
#include <sys/select.h>
|
||||
#include <sys/time.h>
|
||||
|
|
@ -754,7 +755,7 @@ rfb_handle(struct rfb_softc *rc, int cfd)
|
|||
{
|
||||
const char *vbuf = "RFB 003.008\n";
|
||||
unsigned char buf[80];
|
||||
unsigned char *message;
|
||||
unsigned char *message = NULL;
|
||||
|
||||
#ifndef NO_OPENSSL
|
||||
unsigned char challenge[AUTH_LENGTH];
|
||||
|
|
@ -766,7 +767,7 @@ rfb_handle(struct rfb_softc *rc, int cfd)
|
|||
#endif
|
||||
|
||||
pthread_t tid;
|
||||
uint32_t sres;
|
||||
uint32_t sres = 0;
|
||||
int len;
|
||||
|
||||
rc->cfd = cfd;
|
||||
|
|
@ -858,7 +859,7 @@ rfb_handle(struct rfb_softc *rc, int cfd)
|
|||
stream_write(cfd, &sres, 4);
|
||||
|
||||
if (sres) {
|
||||
*((uint32_t *) buf) = htonl(strlen(message));
|
||||
be32enc(buf, strlen(message));
|
||||
stream_write(cfd, buf, 4);
|
||||
stream_write(cfd, message, strlen(message));
|
||||
goto done;
|
||||
|
|
|
|||
Loading…
Reference in a new issue