Flawfinder version 2.0.10, (C) 2001-2019 David A. Wheeler. Number of rules (primarily dangerous function names) in C/C++ ruleset: 223 Examining data/dhcp-probe-1.3.0/src/get_myeaddr.h Examining data/dhcp-probe-1.3.0/src/configfile.h Examining data/dhcp-probe-1.3.0/src/utils.h Examining data/dhcp-probe-1.3.0/src/report.c Examining data/dhcp-probe-1.3.0/src/daemonize.h Examining data/dhcp-probe-1.3.0/src/defs.h Examining data/dhcp-probe-1.3.0/src/open_max.h Examining data/dhcp-probe-1.3.0/src/daemonize.c Examining data/dhcp-probe-1.3.0/src/defaults.h Examining data/dhcp-probe-1.3.0/src/utils.c Examining data/dhcp-probe-1.3.0/src/dhcp_probe.h Examining data/dhcp-probe-1.3.0/src/get_myeaddr.c Examining data/dhcp-probe-1.3.0/src/configfile.c Examining data/dhcp-probe-1.3.0/src/open_max.c Examining data/dhcp-probe-1.3.0/src/report.h Examining data/dhcp-probe-1.3.0/src/dhcp_probe.c Examining data/dhcp-probe-1.3.0/src/get_myipaddr.h Examining data/dhcp-probe-1.3.0/src/bootp.h Examining data/dhcp-probe-1.3.0/src/bootp.c Examining data/dhcp-probe-1.3.0/src/get_myipaddr.c Examining data/dhcp-probe-1.3.0/lib/strerror.c Examining data/dhcp-probe-1.3.0/lib/inet_aton.h Examining data/dhcp-probe-1.3.0/lib/inet_aton.c Examining data/dhcp-probe-1.3.0/lib/my_null_proc.c Examining data/dhcp-probe-1.3.0/lib/strerror.h FINAL RESULTS: data/dhcp-probe-1.3.0/src/configfile.c:126:12: [4] (buffer) sscanf: The scanf() family's %s operation, without a limit specification, permits buffer overflows (CWE-120, CWE-20). Specify a limit to %s, or use a different input function. tokens = sscanf(buf, "%s %s %s %s", str1, str2, str3, str4); data/dhcp-probe-1.3.0/src/dhcp_probe.c:912:8: [4] (shell) execl: This causes a new program to execute and is difficult to use safely (CWE-78). try using a library call that implements the same functionality if available. if (execl(alert_program_name, alert_program_name, prog, ifname, ip_src_str, ether_shost_str, (char *) 0 ) < 0) { data/dhcp-probe-1.3.0/src/dhcp_probe.c:940:16: [4] (shell) execl: This causes a new program to execute and is difficult to use safely (CWE-78). try using a library call that implements the same functionality if available. execl_rc = execl(alert_program_name2, alert_program_name2, "-p", prog, "-I", ifname, "-i", ip_src_str, "-m", ether_shost_str, "-y", yiaddr_str, (char *) 0 ); data/dhcp-probe-1.3.0/src/dhcp_probe.c:943:16: [4] (shell) execl: This causes a new program to execute and is difficult to use safely (CWE-78). try using a library call that implements the same functionality if available. execl_rc = execl(alert_program_name2, alert_program_name2, "-p", prog, "-I", ifname, "-i", ip_src_str, "-m", ether_shost_str, (char *) 0 ); data/dhcp-probe-1.3.0/src/get_myeaddr.c:125:3: [4] (buffer) strcpy: Does not check for buffer overflows when copying to destination [MS-banned] (CWE-120). Consider using snprintf, strcpy_s, or strlcpy (warning: strncpy easily misused). strcpy(arpreq.arp_dev, ifname); data/dhcp-probe-1.3.0/src/report.c:205:2: [4] (format) vsnprintf: If format strings can be influenced by an attacker, they can be exploited, and note that sprintf variations do not always \0-terminate (CWE-134). Use a constant for the format specification. vsnprintf(buf, sizeof(buf), fmt, ap); data/dhcp-probe-1.3.0/src/report.c:207:2: [4] (format) vsprintf: Potential format string problem (CWE-134). Make format string constant. vsprintf(buf, fmt, ap); data/dhcp-probe-1.3.0/src/report.c:238:4: [4] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. sprintf(time_buf, "%02d-%s-%4d %02d:%02d:%02d ", data/dhcp-probe-1.3.0/src/dhcp_probe.c:111:14: [3] (buffer) getopt: Some older implementations do not protect against internal buffer overflows (CWE-120, CWE-20). Check implementation on installation, or limit the size of all string inputs. while ((c = getopt(argc, argv, "c:d:fhl:o:p:Q:s:Tvw:")) != EOF) { data/dhcp-probe-1.3.0/lib/strerror.c:49:8: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. static char errmsg[80]; /* holds "Error %d" */ data/dhcp-probe-1.3.0/lib/strerror.c:57:11: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. return (char *) sys_errlist[en]; /* cast away const since strerror(3) isn't declared that way */ data/dhcp-probe-1.3.0/lib/strerror.c:61:2: [2] (buffer) sprintf: Does not check for buffer overflows (CWE-120). Use sprintf_s, snprintf, or vsnprintf. Risk is low because the source has a constant maximum length. sprintf(errmsg, "Error %d", en); data/dhcp-probe-1.3.0/src/bootp.c:23:10: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. unsigned char vendor_option_clientid[1 + 1 + 1 + sizeof(struct ether_addr)]; /* option code, length byte, htype byte, ether_addr */ data/dhcp-probe-1.3.0/src/bootp.c:24:10: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. unsigned char vendor_option_serverid[1 + 1 + 4]; /* option code, length byte, ip_addr */ data/dhcp-probe-1.3.0/src/bootp.c:25:10: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. unsigned char vendor_option_requestedipaddr[1 + 1 + 4]; /* option code, length byte, ip_addr */ data/dhcp-probe-1.3.0/src/bootp.c:73:3: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char label[NUM_FLAVORS_MAXSTRING]; data/dhcp-probe-1.3.0/src/bootp.c:127:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(GetChaddr(), &packet->bootp_chaddr, sizeof(struct ether_addr)); data/dhcp-probe-1.3.0/src/bootp.c:318:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(GetChaddr(), vendor_option_clientid+3, sizeof(struct ether_addr)); data/dhcp-probe-1.3.0/src/bootp.c:371:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(option, *destination, len); data/dhcp-probe-1.3.0/src/bootp.h:21:11: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. unsigned char bootp_chaddr[BOOTP_CHADDR_LEN]; /* client hardware address */ data/dhcp-probe-1.3.0/src/bootp.h:22:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char bootp_sname[BOOTP_SNAME_LEN]; /* server host name */ data/dhcp-probe-1.3.0/src/bootp.h:23:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char bootp_file[BOOTP_FILE_LEN]; /* boot file name */ data/dhcp-probe-1.3.0/src/bootp.h:24:11: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. unsigned char bootp_options[BOOTP_OPTIONS_LEN]; /* options */ data/dhcp-probe-1.3.0/src/configfile.c:82:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char buf[BUFSIZ]; /* one entire line read from input file */ data/dhcp-probe-1.3.0/src/configfile.c:84:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char str1[BUFSIZ], str2[BUFSIZ], str3[BUFSIZ], str4[BUFSIZ]; /* tokens parsed by sscanf */ data/dhcp-probe-1.3.0/src/configfile.c:115:12: [2] (misc) fopen: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). if ((fp = fopen(fname, "r")) == NULL) { data/dhcp-probe-1.3.0/src/configfile.c:146:4: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(enet, &chaddr, sizeof(chaddr)); data/dhcp-probe-1.3.0/src/configfile.c:165:4: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(enet, ðer_src, sizeof(ether_src)); data/dhcp-probe-1.3.0/src/configfile.c:291:4: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(enet, &legal_server_ethersrcs[num_legal_server_ethersrcs], sizeof(legal_server_ethersrcs[num_legal_server_ethersrcs])); data/dhcp-probe-1.3.0/src/configfile.c:335:5: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char addr_str_tmp[MAX_IP_ADDR_STR]; data/dhcp-probe-1.3.0/src/configfile.c:336:5: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(inet_ntoa(lease_networks_of_concern_addr[num_lease_networks_of_concern]), &addr_str_tmp, sizeof(addr_str_tmp)); data/dhcp-probe-1.3.0/src/configfile.c:436:3: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&chaddr, &chaddr_copy, sizeof(chaddr_copy)); data/dhcp-probe-1.3.0/src/configfile.c:438:3: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&my_eaddr, &chaddr_copy, sizeof(chaddr_copy)); data/dhcp-probe-1.3.0/src/configfile.c:458:3: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(ðer_src, ðer_src_copy, sizeof(ether_src_copy)); data/dhcp-probe-1.3.0/src/configfile.c:460:3: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&my_eaddr, ðer_src_copy, sizeof(ether_src_copy)); data/dhcp-probe-1.3.0/src/defs.h:139:9: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. #define bcopy(a, b, c) memcpy(b, a, c) data/dhcp-probe-1.3.0/src/defs.h:139:25: [2] (buffer) memcpy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. #define bcopy(a, b, c) memcpy(b, a, c) data/dhcp-probe-1.3.0/src/dhcp_probe.c:99:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char pcap_errbuf[PCAP_ERRBUF_SIZE], pcap_errbuf2[PCAP_ERRBUF_SIZE]; data/dhcp-probe-1.3.0/src/dhcp_probe.c:102:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char libnet_errbuf[LIBNET_ERRBUF_SIZE]; data/dhcp-probe-1.3.0/src/dhcp_probe.c:709:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char yiaddr_network_of_concern_addenda[STR_MAXLEN]; data/dhcp-probe-1.3.0/src/dhcp_probe.c:716:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char ether_dhost_str[MAX_ETHER_ADDR_STR], ether_shost_str[MAX_ETHER_ADDR_STR]; data/dhcp-probe-1.3.0/src/dhcp_probe.c:717:2: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char ip_src_str[MAX_IP_ADDR_STR], ip_dst_str[MAX_IP_ADDR_STR], yiaddr_str[MAX_IP_ADDR_STR]; data/dhcp-probe-1.3.0/src/dhcp_probe.c:741:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&(ether_header->ether_dhost), ðer_dhost, sizeof(ether_dhost)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:742:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&(ether_header->ether_shost), ðer_shost, sizeof(ether_shost)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:744:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(ether_ntoa(ðer_dhost), ðer_dhost_str, sizeof(ether_dhost_str)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:745:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(ether_ntoa(ðer_shost), ðer_shost_str, sizeof(ether_shost_str)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:759:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&(ip_header->ip_src), &ip_src, sizeof(ip_header->ip_src)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:760:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&(ip_header->ip_dst), &ip_dst, sizeof(ip_header->ip_dst)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:762:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(inet_ntoa(ip_src), &ip_src_str, sizeof(ip_src_str)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:763:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(inet_ntoa(ip_dst), &ip_dst_str, sizeof(ip_dst_str)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:824:4: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char ether_tmp_str[MAX_ETHER_ADDR_STR]; data/dhcp-probe-1.3.0/src/dhcp_probe.c:827:4: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&(bootp_pkt->bootp_chaddr), ðer_tmp, sizeof(ether_tmp)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:828:4: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(ether_ntoa(ðer_tmp), ðer_tmp_str, sizeof(ether_tmp_str)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:861:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(&(bootp_pkt->bootp_yiaddr), &yiaddr, sizeof(bootp_pkt->bootp_yiaddr)); data/dhcp-probe-1.3.0/src/dhcp_probe.c:863:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(inet_ntoa(yiaddr), &yiaddr_str, sizeof(yiaddr_str)); data/dhcp-probe-1.3.0/src/get_myeaddr.c:95:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(ifr.ifr_hwaddr.sa_data, my_eaddr, sizeof (struct ether_addr)); data/dhcp-probe-1.3.0/src/get_myeaddr.c:118:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(my_ipaddr, &sin->sin_addr, sizeof(struct in_addr)); data/dhcp-probe-1.3.0/src/get_myeaddr.c:132:10: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). if ((fd=open("/dev/arp", O_RDONLY)) < 0) { data/dhcp-probe-1.3.0/src/get_myeaddr.c:157:2: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy(arpreq.arp_ha.sa_data, my_eaddr, sizeof (struct ether_addr)); data/dhcp-probe-1.3.0/src/get_myeaddr.c:183:5: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy((const void *)LLADDR(sdl), my_eaddr, sizeof (struct ether_addr)); data/dhcp-probe-1.3.0/src/get_myipaddr.c:159:3: [2] (buffer) bcopy: Does not check for buffer overflows when copying to destination (CWE-120). Make sure destination can always hold the source data. bcopy((char *) &sip->sin_addr, (char *) &(my_ipaddr->s_addr), sizeof(struct in_addr)); data/dhcp-probe-1.3.0/src/report.c:194:9: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. static char buf[256]; data/dhcp-probe-1.3.0/src/report.c:226:3: [2] (buffer) char: Statically-sized arrays can be improperly restricted, leading to potential overflows or other issues (CWE-119!/CWE-120). Perform bounds checking, use functions that limit length, or ensure that the size is larger than the maximum possible length. char time_buf[256]; data/dhcp-probe-1.3.0/src/utils.c:78:7: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). fd = open(filename, O_WRONLY|O_CREAT|O_EXCL, S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH); data/dhcp-probe-1.3.0/src/utils.c:134:7: [2] (misc) open: Check when opening files - can an attacker redirect it (via symlinks), force the opening of special file type (e.g., device files), move things around to create a race condition, control its ancestors, or change its contents? (CWE-362). fd = open(filename, O_WRONLY|O_APPEND|O_CREAT, S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH); data/dhcp-probe-1.3.0/src/get_myeaddr.c:89:2: [1] (buffer) strncpy: Easily used incorrectly; doesn't always \0-terminate or check for invalid pointers [MS-banned] (CWE-120). strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name)); data/dhcp-probe-1.3.0/src/get_myeaddr.c:176:8: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). if ((strlen(ifname) == strlen(ifp->ifa_name)) && !strcmp(ifp->ifa_name, ifname)) { data/dhcp-probe-1.3.0/src/get_myeaddr.c:176:26: [1] (buffer) strlen: Does not handle strings that are not \0-terminated; if given one it may perform an over-read (it could cause a crash if unprotected) (CWE-126). if ((strlen(ifname) == strlen(ifp->ifa_name)) && !strcmp(ifp->ifa_name, ifname)) { ANALYSIS SUMMARY: Hits = 68 Lines analyzed = 3937 in approximately 0.14 seconds (28050 lines/second) Physical Source Lines of Code (SLOC) = 2305 Hits@level = [0] 37 [1] 3 [2] 56 [3] 1 [4] 8 [5] 0 Hits@level+ = [0+] 105 [1+] 68 [2+] 65 [3+] 9 [4+] 8 [5+] 0 Hits/KSLOC@level+ = [0+] 45.5531 [1+] 29.5011 [2+] 28.1996 [3+] 3.90456 [4+] 3.47072 [5+] 0 Dot directories skipped = 1 (--followdotdir overrides) Minimum risk level = 1 Not every hit is necessarily a security vulnerability. There may be other security vulnerabilities; review your code! See 'Secure Programming HOWTO' (https://dwheeler.com/secure-programs) for more information.