aboutsummaryrefslogtreecommitdiff
path: root/sys/netgraph/ng_parse.c
Commit message (Collapse)AuthorAgeFilesLines
* sys: Remove $FreeBSD$: one-line .h patternWarner Losh2023-08-161-1/+0
| | | | Remove /^\s*\*+\s*\$FreeBSD\$.*$\n/
* ng_parse: IP address parsing in netgraph eating too many charactersMarkus Stoff2021-05-181-3/+5
| | | | | | | | | | | Once the final component of the IP address has been parsed, the offset on the input must not be advanced, as this would remove an unparsed character from the input. Submitted by: Markus Stoff Reviewed by: donner MFC after: 3 weeks Differential Revision: https://reviews.freebsd.org/D26489
* net: clean up empty lines in .c and .h filesMateusz Guzik2020-09-011-2/+0
| | | | Notes: svn path=/head/; revision=365071
* Revert r327828, r327949, r327953, r328016-r328026, r328041:Pedro F. Giffuni2018-01-211-2/+1
| | | | | | | | | | | | | | | | | | Uses of mallocarray(9). The use of mallocarray(9) has rocketed the required swap to build FreeBSD. This is likely caused by the allocation size attributes which put extra pressure on the compiler. Given that most of these checks are superfluous we have to choose better where to use mallocarray(9). We still have more uses of mallocarray(9) but hopefully this is enough to bring swap usage to a reasonable level. Reported by: wosch PR: 225197 Notes: svn path=/head/; revision=328218
* netgraph: make some use of mallocarray(9).Pedro F. Giffuni2018-01-151-1/+2
| | | | | | | | | | | | | | | | Focus on code where we are doing multiplications within malloc(9). None of these ire likely to overflow, however the change is still useful as some static checkers can benefit from the allocation attributes we use for mallocarray. This initial sweep only covers malloc(9) calls with M_NOWAIT. No good reason but I started doing the changes before r327796 and at that time it was convenient to make sure the sorrounding code could handle NULL values. X-Differential revision: https://reviews.freebsd.org/D13837 Notes: svn path=/head/; revision=328024
* Fix a bunch of -Wcast-qual warnings in netgraph's ng_parse.c, by usingDimitry Andric2015-01-301-3/+3
| | | | | | | | | __DECONST. No functional change. MFC after: 3 days Notes: svn path=/head/; revision=277951
* Fix compilation warning.Gleb Smirnoff2013-02-151-2/+4
| | | | | | | Sponsored by: Nginx, Inc Notes: svn path=/head/; revision=246821
* Mark MALLOC_DEFINEs static that have no corresponding MALLOC_DECLAREs.Ed Schouten2011-11-071-1/+1
| | | | | | | This means that their use is restricted to a single C file. Notes: svn path=/head/; revision=227293
* remove RESTARTABLE_PANICS optionAndriy Gapon2011-07-251-12/+0
| | | | | | | | | | | | | | | | | | | This is done per request/suggestion from John Baldwin who introduced the option. Trying to resume normal system operation after a panic is very unpredictable and dangerous. It will become even more dangerous when we allow a thread in panic(9) to penetrate all lock contexts. I understand that the only purpose of this option was for testing scenarios potentially resulting in panic. Suggested by: jhb Reviewed by: attilio, jhb X-MFC-After: never Approved by: re (kib) Notes: svn path=/head/; revision=224307
* Retire the MALLOC and FREE macros. They are an abomination unto style(9).Dag-Erling Smørgrav2008-10-231-21/+21
| | | | | | | MFC after: 3 months Notes: svn path=/head/; revision=184205
* Fix various compilation warnings for gcc-4.2.Matt Jacob2007-06-231-3/+3
| | | | | | | Approved by: re (bruce) Notes: svn path=/head/; revision=170996
* Fix two accesses to uninitialized variables that a revision 1.27Ruslan Ermilov2006-01-181-4/+8
| | | | | | | | | has introduced. Found with: Coverity Prevent(tm) Notes: svn path=/head/; revision=154521
* Make code simplier fixing memory leak.Gleb Smirnoff2006-01-141-5/+2
| | | | | | | Found with: Coverity Prevent(tm) Notes: svn path=/head/; revision=154375
* Fix two memory leakages.Gleb Smirnoff2006-01-141-4/+5
| | | | | | | Found with: Coverity Prevent(tm) Notes: svn path=/head/; revision=154353
* Fixed parsing of unsigned integers.Ruslan Ermilov2005-08-021-1/+4
| | | | Notes: svn path=/head/; revision=148645
* Replace NG_PARSE_APPEND() macro with ng_parse_append() function. CheckGleb Smirnoff2005-03-011-27/+70
| | | | | | | | | | | its return value and free resources if function returns error. Plug several memory leaks with this change. Submitted by: archie Found by: Coverity Prevent analysis tool Notes: svn path=/head/; revision=142902
* /* -> /*- for license, minor formatting changesWarner Losh2005-01-071-2/+3
| | | | Notes: svn path=/head/; revision=139823
* Avoid casts as lvalues.Alexander Kabaev2004-07-281-3/+3
| | | | Notes: svn path=/head/; revision=132780
* Convert Netgraph to use mbuf tags to pass its meta information around.Julian Elischer2004-06-251-0/+1
| | | | | | | | | | | Thanks to Sam for importing tags in a way that allowed this to be done. Submitted by: Gleb Smirnoff <glebius@cell.sick.ru> Also allow the sr and ar drivers to create netgraph versions of their modules. Document the change to the ksocket node. Notes: svn path=/head/; revision=131108
* Fix a memory leak in ng_get_string_token. A dynamically-allocatedJohn Polstra2004-04-291-0/+1
| | | | | | | | | buffer wasn't freed if the function failed. MFC after: 3 days Notes: svn path=/head/; revision=128729
* Get rid of the deprecated *LEN constants in favour of the newHartmut Brandt2004-01-261-5/+5
| | | | | | | *SIZ constants that include the trailing \0 byte. Notes: svn path=/head/; revision=125028
* Made the Ethernet address parse type standard.Ruslan Ermilov2003-12-171-0/+58
| | | | | | | OK'ed by: archie Notes: svn path=/head/; revision=123600
* Use intptr_t to fix various sizeof(int) != sizeof(void *) warnings.John Baldwin2002-11-081-4/+4
| | | | Notes: svn path=/head/; revision=106665
* Fix GCC warnings caused by initializing a zero length array. In the process,Archie Cobbs2002-05-311-17/+19
| | | | | | | | | | simply things a bit by getting rid of 'struct ng_parse_struct_info' which was useless because it only contained one field. MFC after: 2 weeks Notes: svn path=/head/; revision=97685
* Fix warnings: deprecated use of label at end of compound statementPeter Wemm2002-05-241-1/+1
| | | | Notes: svn path=/head/; revision=97229
* Fix bug in previous commit.Archie Cobbs2002-02-121-2/+1
| | | | | | | Submitted by: Harti Brandt <brandt@fokus.gmd.de> Notes: svn path=/head/; revision=90584
* Some netgraph parse types (such as for the 'value' field in ng_ksocket'sArchie Cobbs2002-02-011-8/+15
| | | | | | | | | | | 'struct ng_ksocket_sockopt') like to peek into the ng_mesg header for information. Make sure when generating default values that we provide a valid header to peek into. MFC after: 1 week Notes: svn path=/head/; revision=90047
* Update to C99, s/__FUNCTION__/__func__/,David E. O'Brien2001-12-101-7/+7
| | | | | | | also don't use ANSI string concatenation. Notes: svn path=/head/; revision=87599
* KSE Milestone 2Julian Elischer2001-09-121-0/+12
| | | | | | | | | | | | | | | | | Note ALL MODULES MUST BE RECOMPILED make the kernel aware that there are smaller units of scheduling than the process. (but only allow one thread per process at this time). This is functionally equivalent to teh previousl -current except that there is a thread associated with each process. Sorry john! (your next MFC will be a doosie!) Reviewed by: peter@freebsd.org, dillon@freebsd.org X-MFC after: ha ha ha ha Notes: svn path=/head/; revision=83366
* Fix a range checking bug in ng_int32_parse which affected 64-bitJohn Polstra2001-05-191-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | machines. The code formerly read: long val; if (val < (long)-0x80000000 || ...) return EINVAL; The constant 0x80000000 has type unsigned int. The unary `-' operator does not change the type (or the value, in this case). Therefore the promotion to long is done by 0-extension, giving 0x0000000080000000 instead of the desired 0xffffffff80000000. I got rid of the `-' and changed the cast to (int32_t) to give proper sign-extension on all architectures and to better reflect the fact that we are range-checking a 32-bit value. This commit also makes the analogous changes to ng_int{8,16}_parse for consistency. MFC after: 3 days Notes: svn path=/head/; revision=76860
* Fix some memory leaksJulian Elischer2001-01-101-16/+23
| | | | | | | Add memory leak detection assitance. Notes: svn path=/head/; revision=70870
* Add the use of M_ZERO to netgraph.David Malone2000-11-181-2/+1
| | | | | | | | | | Submitted by: josh@zipperup.org Submitted by: Robert Drehmel <robd@gmx.net> Submitted by: archie Approved by: archie Notes: svn path=/head/; revision=68876
* Go back to using data_len in struct ngpppoe_init_data after discussionsBrian Somers2000-11-161-19/+86
| | | | | | | | | | | | | | with Julian and Archie. Implement a new ``sizedstring'' parse type for dealing with field pairs consisting of a uint16_t followed by a data field of that size, and use this to deal with the data_len and data fields. Written by: Archie with some input by me Agreed in principle by: julian Notes: svn path=/head/; revision=68845
* Since neither archie nor I work at Whistle any more, change our emailJulian Elischer2000-10-241-1/+1
| | | | | | | | | addresses to be the more usefu @freebsd.org ones so we can keep getting bug-reports. - man pages to follow.. Notes: svn path=/head/; revision=67506
* Avoid free'ing a NULL pointer.Archie Cobbs2000-08-311-1/+2
| | | | Notes: svn path=/head/; revision=65303
* - Add new unsigned and hex integer parse types; this allows simplifyingArchie Cobbs2000-08-101-41/+146
| | | | | | | | | the bytearray parse type. - Allocate (larger) temporary work buffer dynamically instead of on the stack when comparing to the default value. Notes: svn path=/head/; revision=64505
* Fix typo: "ng_parse_fixedsstring_info" -> "ng_parse_fixedstring_info"Archie Cobbs2000-03-131-8/+8
| | | | Notes: svn path=/head/; revision=58011
* Remove a bunch of un-needed includes.Julian Elischer1999-12-071-2/+0
| | | | | | | Submitted by: phk@freebsd.org Notes: svn path=/head/; revision=54249
* Fix bug parsing 32 bit integers on machines where sizeof(long) == 4.Archie Cobbs1999-12-031-1/+2
| | | | Notes: svn path=/head/; revision=54094
* Add two new generic control messages, NGM_ASCII2BINARY andArchie Cobbs1999-11-301-0/+1604
NGM_BINARY2ASCII, which convert control messages to ASCII and back. This allows control messages to be sent and received in ASCII form using ngctl(8), which makes ngctl a lot more useful. This also allows all the type-specific debugging code in libnetgraph to go away -- instead, we just ask the node itself to do the ASCII translation for us. Currently, all generic control messages are supported, as well as messages associated with the following node types: async, cisco, ksocket, and ppp. See /usr/share/examples/netgraph/ngctl for an example of using this. Also give ngctl(8) the ability to print out incoming data and control messages at any time. Eventually nghook(8) may be subsumed. Several other misc. bug fixes. Reviewed by: julian Notes: svn path=/head/; revision=53913