Check-in [846a8770cd]
Overview
SHA1:846a8770cdfbacc529b2339f84b45be598e99046
Date: 2009-03-30 19:25:31
User: rmiller
Comment:Fix to the: packetbl.c: In function ?\226?\128?\152pbl_set_verdict?\226?\128?\153: packetbl.c:593: warning: passing argument 1 of ?\226?\128?\152nfq_set_verdict?\226?\128?\153 discards qualifiers from pointer target type packetbl.c:595: warning: passing argument 1 of ?\226?\128?\152nfq_set_verdict?\226?\128?\153 discards qualifiers from pointer target type errors. Hopefully this doesn't break anything.
Timelines: family | ancestors | descendants | both | trunk
Downloads: Tarball | ZIP archive
Other Links: files | file ages | folders | manifest
Tags And Properties
Context
2009-05-10
18:10
[8ad0406dfd] Patch submitted by Julien Reveret that makes packetbl actually work under 64 bit systems. (user: rmiller, tags: trunk)
2009-03-30
19:25
[846a8770cd] Fix to the: packetbl.c: In function ?\226?\128?\152pbl_set_verdict?\226?\128?\153: packetbl.c:593: warning: passing argument 1 of ?\226?\128?\152nfq_set_verdict?\226?\128?\153 discards qualifiers from pointer target type packetbl.c:595: warning: passing argument 1 of ?\226?\128?\152nfq_set_verdict?\226?\128?\153 discards qualifiers from pointer target type errors. Hopefully this doesn't break anything. (user: rmiller, tags: trunk)
19:17
[724909d7b7] Use conf instead of config. For some reason, this works for me, but appears to not work on other systems. (user: rmiller, tags: trunk)
Changes

Modified packetbl.c from [72cafc699e] to [047b0a35f3].

   177    177   int check_packet_dnsbl(const struct packet_info *ip, struct config_entry *list);
   178    178   int parse_cidr(struct config_entry *ce);
   179    179   /* int validate_blacklist(char *); */
   180    180   void parse_config(void);
   181    181   void parse_arguments(int argc, char **argv);
   182    182   void pbl_init_sockstat(void);
   183    183   static void get_ip_string(const struct packet_info *ip);
   184         -static void pbl_set_verdict(const struct PBL_HANDLE *h, PBL_ID_T id,
          184  +static void pbl_set_verdict(struct PBL_HANDLE *h, PBL_ID_T id,
   185    185           unsigned int verdict);
   186    186   
   187    187   static int pbl_callback(struct nfq_q_handle *qh, struct nfgenmsg *nfmsg,
   188    188   	struct nfq_data *nfa, void *data);
   189    189   	
   190    190   static const configoption_t options[] = {
   191    191   	{"<host>", ARG_NONE, host_section_open, NULL, O_ROOT},
................................................................................
   582    582    * NOTES:
   583    583    *   This function calls ipq_set_verdict() to the appropriate "verdict"
   584    584    *   It must be able to handle the condition where "conf.dryrun" is set
   585    585    *   causing all "verdict" values to be treated as NF_ACCEPT regardless
   586    586    *   of their actual value.
   587    587    *
   588    588    */
   589         -static void pbl_set_verdict(const struct PBL_HANDLE *h, PBL_ID_T id,
          589  +static void pbl_set_verdict(struct PBL_HANDLE *h, PBL_ID_T id,
   590    590   	unsigned int verdict) {
   591    591   
   592    592   	if (conf.dryrun == 1) {
   593    593   		SET_VERDICT(h, id, NF_ACCEPT, 0, NULL);
   594    594   	} else {
   595    595   		SET_VERDICT(h, id, verdict, 0, NULL);
   596    596   	}