summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fixed spelling erorrs.HEADmasterThomas Osterried2023-05-138-9/+9
| | | | Signed-off-by: Thomas Osterried <dl9sau@darc.de>
* Fixed spelling errors in man pages.Thomas Osterried2023-05-138-13/+13
| | | | | | | Thanks to Danielle Forsi Signed-off-by: Daniele Forsi <iu5hkx@gmail.com> Signed-off-by: Thomas Osterried <dl9sau@darc.de>
* Hint for first time compilationThomas Osterried2022-09-161-0/+12
| | | | Signed-off-by: Thomas Osterried <ax25@x-berg.in-berlin.de>
* fixed typo in man pageThomas Osterried2022-05-271-1/+1
| | | | Signed-off-by: Thomas Osterried <ax25@x-berg.in-berlin.de>
* Moved setsockopt section behind bind().Thomas Osterried2022-05-131-16/+21
| | | | | | | | | Reason: kernel overwrites general and device specific settings after bind(). Thus all setsockopt-changes will be lost. Newer kernels should notice users about this. Signed-off-by: Thomas Osterried <ax25@x-berg.in-berlin.de>
* In setsockopt section:Thomas Osterried2022-05-021-10/+10
| | | | | | | | | | | if user requests EAX25 (option -me) and specifies a window size (> 7), call failed, because kernel checks if socket is in eax25 mode when setting window size > 7. -> setsockopt(...AX25_EXTSEQ...) has to be called before setsockopt(...AX25_WINDOW...) => Moved the etsockopt(...AX25_EXTSEQ...) in the code part in front of setsockopt(...AX25_WINDOW...) Signed-off-by: Thomas Osterried <ax25@x-berg.in-berlin.de>
* Added line delimiter before '-m' paragraph.Thomas Osterried2022-05-021-0/+1
| | | | Signed-off-by: Thomas Osterried <ax25@x-berg.in-berlin.de>
* ax25ipd: Fix warning and potential buffer overflow in bpqether.c.Ralf Baechle2021-07-281-2/+13
| | | | | | | | | | | | | | | | Adding an additional check convinces GCC 11 there won't be a buffer overflow. Reading the code it's pretty hard to convince myself overflow is impossible so just suck it up and throw a potentially pointless check into the spaghetti. gcc -DHAVE_CONFIG_H -I. -I.. -DAX25_SYSCONFDIR=\""/usr/local/etc/ax25"\" -DAX25_LOCALSTATEDIR=\""/usr/local/var/ax25"\" -g -O2 -Wall -MT bpqether.o -MD -MP -MF .deps/bpqether.Tpo -c -o bpqether.o bpqether.c In function ‘tun_alloc’, inlined from ‘open_ethertap’ at bpqether.c:196:8: bpqether.c:142:17: warning: ‘strncpy’ output may be truncated copying 15 bytes from a string of length 4095 [-Wstringop-truncation] 142 | strncpy(ifr.ifr_name, dev, IFNAMSIZ); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Include <stdlib.h> for malloc.Ralf Baechle2021-07-281-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | This fixes a warning about implicit declaration of malloc. gcc -DHAVE_CONFIG_H -I. -I.. -DAX25_SYSCONFDIR=\""/usr/local/etc/ax25"\" -DAX25_LOCALSTATEDIR=\""/usr/local/var/ax25"\" -g -O2 -Wall -MT routing.o -MD -MP -MF .deps/routing.Tpo -c -o routing.o routing.c routing.c: In function ‘route_add’: routing.c:83:13: warning: implicit declaration of function ‘malloc’ [-Wimplicit-function-declaration] 83 | malloc(sizeof(struct route_table_entry)); | ^~~~~~ routing.c:17:1: note: include ‘<stdlib.h>’ or provide a declaration of ‘malloc’ 16 | #include "ax25ipd.h" +++ |+#include <stdlib.h> 17 | routing.c:83:13: warning: incompatible implicit declaration of built-in function ‘malloc’ [-Wbuiltin-declaration-mismatch] 83 | malloc(sizeof(struct route_table_entry)); | ^~~~~~ routing.c:83:13: note: include ‘<stdlib.h>’ or provide a declaration of ‘malloc’ routing.c: In function ‘bcast_add’: routing.c:130:13: warning: incompatible implicit declaration of built-in function ‘malloc’ [-Wbuiltin-declaration-mismatch] 130 | malloc(sizeof(struct bcast_table_entry)); | ^~~~~~ routing.c:130:13: note: include ‘<stdlib.h>’ or provide a declaration of ‘malloc’ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* listen: Fix potential buffer overflow.Ralf Baechle2021-07-281-3/+3
| | | | | | | | | | | | | | | | | | | | Sounds evil but would only be exploitable when getservbyport is accessing a database that's under an attacker's control. The /etc/services file shipping with Fedora 34 has entries of a maximum length of 15 characters so are just fine. Fixed by not copying the string returned by getservbyport - which also is faster. At that point it becomes possible to reduce the length of str[] to 6 bytes. Found by the following GCC 11 warning: gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT utils.o -MD -MP -MF .deps/utils.Tpo -c -o utils.o utils.c utils.c: In function ‘servname’: utils.c:124:17: warning: ‘strncpy’ specified bound 16 equals destination size [-Wstringop-truncation] 124 | strncpy(str, serv->s_name, 16); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* listen: Fix overzealous GCC warnings.Ralf Baechle2021-07-281-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GCC 11 has returned to the old tradition of overzealous warnings. Rewrite the conversion of time in seconds to hours, minutes and seconds to something that not only fixes the warnings but also is more readable. gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT listen.o -MD -MP -MF .deps/listen.Tpo -c -o listen.o listen.c listen.c: In function ‘display_timestamp’: listen.c:73:52: warning: ‘%06u’ directive output may be truncated writing 6 bytes into a region of size between 2 and 7 [-Wformat-truncation=] 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~ listen.c:73:36: note: directive argument in the range [0, 999999] 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~~~~~~~~~~~~~~~~~~ listen.c:73:9: note: ‘snprintf’ output between 16 and 21 bytes into a destination of size 16 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 74 | hours, minutes, seconds, usec); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ listen.c:73:52: warning: ‘%06u’ directive output may be truncated writing 6 bytes into a region of size between 2 and 7 [-Wformat-truncation=] 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~ listen.c:73:36: note: directive argument in the range [0, 999999] 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~~~~~~~~~~~~~~~~~~ listen.c:73:9: note: ‘snprintf’ output between 16 and 21 bytes into a destination of size 16 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 74 | hours, minutes, seconds, usec); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ listen.c:73:52: warning: ‘%06u’ directive output may be truncated writing 6 bytes into a region of size between 2 and 7 [-Wformat-truncation=] 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~ listen.c:73:36: note: directive argument in the range [0, 999999] 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~~~~~~~~~~~~~~~~~~ listen.c:73:9: note: ‘snprintf’ output between 16 and 21 bytes into a destination of size 16 73 | snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 74 | hours, minutes, seconds, usec); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* Listen: Fix test for hours in proper value range.Ralf Baechle2021-07-281-1/+1
| | | | | | | | | This really was a hint to GCC to know the value range so it spit bogus warnings so this is not a functional fix. And of course hours are restricted to values of 0..23, not 0..59. Signed-off-by: Ralf Baechle <ralf@linux-mips.org> Fixes: 6553ae600526 ("listen: Fix overzealous GCC 7 compiler warnings.")
* Fix spelling of Klaus Kudielka OE1KIB's first name.Ralf Baechle2021-07-284-4/+4
| | | | | | | | | The misspelling was first introduced in ax25-apps 0.0.3 from 1999-08-10, then copied around the tree. Sorry Klaus! Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* configure: Paranoidly quote things for m4 macro expansion safety.Ralf Baechle2019-04-111-2/+2
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* libtool: Remove useless libtool support.Ralf Baechle2019-04-113-5/+0
| | | | | | | | ax25-apps doesn't build any libraries, so libtools support is useless. Historically libtools was enabled in the ax25 build system since 0.0.1, the oldest release still in existence. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* configure: Remove AC_TYPE_SIGNAL.Ralf Baechle2019-04-111-1/+0
| | | | | | | AC_TYPE_SIGNAL only made sense for some SysV R3 crap which is only rotten and (almost) forgotten. We also don't support it. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* treewide: Update all references to the FSF's address.Ralf Baechle2019-04-118-8/+16
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* COPYING: Update license from FSF source.Ralf Baechle2019-04-101-21/+20
| | | | | | | | The checked in version is an unchanged version of the file obtained from https://www.gnu.org/licenses/old-licenses/lgpl-2.1.txt. Aside from minor whitespace changes it fixes the FSF's address which has changed many years ago and reflects the FSF renaming the Library GNU Public License to the GNU Lesser Public License. The actual licensing terms are unchanged.
* Bump version to 0.0.8-rc5.ax25-apps-0.0.8-rc5Ralf Baechle2019-04-021-1/+1
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ChangeLog: Update with changes since ax25-apps-0.0.8-rc4.Ralf Baechle2019-04-021-1/+10
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* RPM: Remove deprecated Group tag from spec file.Ralf Baechle2019-04-021-1/+0
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* RPM: Drop useless %defattr directives.Ralf Baechle2019-04-021-1/+0
| | | | | | They only enforce what's already the default. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25rtd: Fix mismatch between code and documentation.Ralf Baechle2019-03-274-13/+13
| | | | | | | | Documentation and some comments were refering to /var/ax25/ax25rtd/- ax25_routes rsp. /var/ax25/ax25rtd/ip_routes while the actual code was using ax25_route rsp. ip_route, that is without plural 's'. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* RPM: Install call as axcall and listen as axlisten.Ralf Baechle2019-03-261-1/+1
| | | | | | | | | | This is already standard with distributions since several years so change the rpm spec file to follow. For the time being the default names when using the configure script will however stay unchanged. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* RPM: Add a few BuildRequires: Tags as noticed in mock builds.Ralf Baechle2019-03-261-0/+3
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* RPM: Add distribution version to rpm release tag.Ralf Baechle2019-03-261-1/+1
| | | | | | This is required by the Fedora versioning guidelines. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* RPM: Throw in a `.' between BASEVERSION and EXTRAVERSION.Ralf Baechle2019-03-261-1/+1
| | | | | | As required by Fedora versioning guidelines. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* autotools: silence annoying autoreconf messages.Ralf Baechle2019-03-203-1/+3
| | | | | | | | | | | | | | | | | | | | | | $ autoreconf --install --force libtoolize: putting auxiliary files in '.'. libtoolize: copying file './ltmain.sh' libtoolize: Consider adding 'AC_CONFIG_MACRO_DIRS([m4])' to configure.ac, libtoolize: and rerunning libtoolize and aclocal. libtoolize: Consider adding '-I m4' to ACLOCAL_AMFLAGS in Makefile.am. configure.ac:10: installing './compile' configure.ac:14: installing './config.guess' configure.ac:14: installing './config.sub' configure.ac:4: installing './install-sh' configure.ac:4: installing './missing' Makefile.am: installing './INSTALL' ax25ipd/Makefile.am: installing './depcomp' $ Silenced down to the normal noise level by adding AC_CONFIG_MACRO_DIRS and ACLOCAL_AMFLAGS as suggested. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* Call: Remove duplicate #include.Ralf Baechle2017-08-081-1/+0
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* call: Remove unused arguments from sevenplname()Ralf Baechle2017-08-031-5/+2
| | | | | | | | | | | | | | | | This fixes the following warnings: gcc -DHAVE_CONFIG_H -I. -I.. -O2 -Wall -Wunused-parameter -flto -MT call.o -MD -MP -MF .deps/call.Tpo -c -o call.o call.c [...] call.c: In function ‘sevenplname’: call.c:1910:52: warning: unused parameter ‘buf’ [-Wunused-parameter] int *logfile, char parms[], int parmsbytes, char buf[], ^~~ call.c:1911:7: warning: unused parameter ‘bytes’ [-Wunused-parameter] int bytes) ^~~~~ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* listen: Remove unnecessary including of <netinet/in.h>Ralf Baechle2017-08-031-1/+0
| | | | | | Broken out from a patch by Wietse Ruyg <pe1oez@dds.nl>. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* Listen: Remove useless postincrement op.Ralf Baechle2017-08-031-1/+1
| | | | | | | GCC would have done that itself but now the code is slightly less eyesore. Broken out from a patch by Wietse Ruyg <pe1oez@dds.nl>. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* treewide: Kill assignments in if conditions.Ralf Baechle2017-08-0310-99/+136
| | | | | | | | | | | | | | | | | | | Somewhat hard to read and the code base already has many overlong lines Found with below spatch files: @parens@ expression E, F, G; binary operator X; statement S; @@ - if ((E = F) X G) + E = F; + if (E X G) S Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Limit SEE ALSO section of man page to things actually related.Ralf Baechle2017-07-281-3/+0
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* Listen: Remove useless parens around argument of return statement.Ralf Baechle2017-07-281-1/+1
| | | | | | | | | | | | | | | Found by below coccinelle script. @parens@ expression E; @@ return - ( E - ) ; Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* listen: Remove all uses of register keyword.Ralf Baechle2017-07-281-4/+4
| | | | | | Compilers are ignoring it for over a quarter century. Time to catch up! Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25rtd: Mention all files used in files section of man page.Ralf Baechle2017-07-281-0/+8
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25rtd: Spelling fix: recomment -> recommendRalf Baechle2017-07-282-2/+2
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* call: Fix structure mismatchRalf Baechle2017-07-274-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | GCC with LTO enabled warn: /bin/sh ../libtool --tag=CC --mode=link gcc -O2 -Wall -flto -o call call.o menu.o crc.o yapp.o dostime.o -lncursesw -lax25 libtool: link: gcc -O2 -Wall -flto -o call call.o menu.o crc.o yapp.o dostime.o -lncursesw -lax25 menu.h:22:9: warning: type of 'winopen' does not match original declaration [-Wlto-type-mismatch] WINDOW* winopen(wint*, int, int, int, int, int); ^ menu.c:20:9: note: 'winopen' was previously declared here WINDOW *winopen(wint * wtab, int nlines, int ncols, int begin_y, ^ menu.c:20:9: note: code may be misoptimized unless -fno-strict-aliasing is used This was caused by different .c files being built with different libc feature selection macros. Adding #define _DEFAULT_SOURCE #define _XOPEN_SOURCE #define _XOPEN_SOURCE_EXTENDED to menu.c would have been sufficient but it seems good practice to add the same definition to all .c files. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* call: Fix possible buffer overflow.Ralf Baechle2017-07-261-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | When downloading a file with a very long filename the 80 byte buffer used by start_ab_download() might overflow. Increase the buffer to a sufficient size and switch to using snprintf for double safety. This also fixes the following warnings: gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT call.o -MD -MP -MF .deps/call.Tpo -c -o call.o call.c call.c: In function ‘cmd_call’: call.c:1008:33: warning: ‘%s’ directive writing up to 254 bytes into a region of size 62 [-Wformat-overflow=] sprintf(s, "filename : %s", gp->file_name); ^~ call.c:1008:3: note: ‘sprintf’ output between 19 and 273 bytes into a destination of size 80 sprintf(s, "filename : %s", gp->file_name); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ call.c:1022:30: warning: ‘%s’ directive writing up to 254 bytes into a region of size 65 [-Wformat-overflow=] sprintf(s, "Unable to open %s", gp->file_name); ^~ call.c:1022:3: note: ‘sprintf’ output between 16 and 270 bytes into a destination of size 80 sprintf(s, "Unable to open %s", gp->file_name); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* listen: Fix overzealous GCC 7 compiler warnings.Ralf Baechle2017-07-262-6/+46
| | | | | | | | | | | | | | | | | | | | | | | make[2]: Entering directory '/home/ralf/src/ax25/ax25-apps/listen' gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT listen.o -MD -MP -MF .deps/listen.Tpo -c -o listen.o listen.c listen.c: In function ‘ts_format’: listen.c:53:58: warning: ‘%06u’ directive output may be truncated writing between 6 and 10 bytes into a region of size between 0 and 7 [-Wformat-truncation=] (void)snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", ^~~~ listen.c:53:42: note: using the range [0, 4294967295] for directive argument (void)snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", ^~~~~~~~~~~~~~~~~~~~~ listen.c:53:9: note: ‘snprintf’ output between 16 and 27 bytes into a destination of size 16 (void)snprintf(buf, sizeof(buf), "%02d:%02d:%02d.%06u", ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ sec / 3600, (sec % 3600) / 60, sec % 60, usec); ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ The value of the variables is such that there is no overflow possible but GCC is not able to figure this out. Change the code such that GCC can figure out the value range thus getting us rid of the variables. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* call: Fix buffer overflow.Ralf Baechle2017-07-261-1/+1
| | | | | | | | | | | | | | | | | | | | | When sending files of 1GB or larger, size_buffer was not large enough to hold the number including a trailing \0. This also fixes the following warning: gcc -DHAVE_CONFIG_H -I. -I.. -g -O2 -Wall -MT yapp.o -MD -MP -MF .deps/yapp.Tpo -c -o yapp.o yapp.c yapp.c: In function ‘yapp_upload_data’: yapp.c:200:24: warning: ‘%ld’ directive writing between 1 and 11 bytes into a region of size 10 [-Wformat-overflow=] sprintf(size_buffer, "%ld", length); ^~~ yapp.c:200:23: note: directive argument in the range [-2147483648, 2147483647] sprintf(size_buffer, "%ld", length); ^~~~~ yapp.c:200:2: note: ‘sprintf’ output between 2 and 12 bytes into a destination of size 10 sprintf(size_buffer, "%ld", length); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Fix printing of version number.Ralf Baechle2017-07-261-1/+1
| | | | | | | | | | fb9187cbfcbe ("ax25ipd: Switch version number printed to ax25-apps version.") removed the VERS2 argument from printf but forgot to remove the %s format string as well potencially crashing or printing garbage when printing the version number. Fixes: fb9187cbfcbe ("ax25ipd: Switch version number printed to ax25-apps version.") Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* call: Fix use of glibc feature selection macrosRalf Baechle2017-07-261-2/+4
| | | | | | | | __USE_XOPEN should not be used directly by application code. Other feature selection macros must be defined before inclusion of any system header files. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Fix glibc feature selection macros.Ralf Baechle2017-07-261-0/+2
| | | | | | | | | | | | | | | | | | | | This fixes the following warnings: gcc -DHAVE_CONFIG_H -I. -I.. -DAX25_SYSCONFDIR=\""/usr/local/etc/ax25"\" -DAX25_LOCALSTATEDIR=\""/usr/local/var/ax25"\" -g -O2 -Wall -MT io.o -MD -MP -MF .deps/io.Tpo -c -o io.o io.c io.c: In function ‘io_open’: io.c:324:18: warning: implicit declaration of function ‘ptsname’; did you mean ‘ttyname’? [-Wimplicit-function-declaration] if ((namepts = ptsname(ttyfd)) == NULL) { ^~~~~~~ ttyname io.c:324:16: warning: assignment makes pointer from integer without a cast [-Wint-conversion] if ((namepts = ptsname(ttyfd)) == NULL) { ^ io.c:329:7: warning: implicit declaration of function ‘unlockpt’; did you mean ‘unlinkat’? [-Wimplicit-function-declaration] if (unlockpt(ttyfd) == -1) { ^~~~~~~~ unlinkat Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Cleanup header file inclusion.Ralf Baechle2017-07-267-54/+52
| | | | | | | | | | We were including way too many system header files, some even multiple times and it wasn't immediately visible. Clean this. This also drops the definitions of __USE_XOPEN which never had the desired effect with glibc. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Replace bzero use with memset.Ralf Baechle2017-07-251-4/+4
| | | | | | bzero is a mega-ancient use of bsdism. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* Compress multiple blank lines into single lines.Ralf Baechle2017-07-2513-31/+0
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Remove code for ICMP mode.Ralf Baechle2017-07-251-44/+0
| | | | | | Supposedly it didn't work and it was never being built anyway. Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
* ax25ipd: Remove USE_SGTTY ifdefery.Ralf Baechle2017-07-251-16/+0
| | | | Signed-off-by: Ralf Baechle <ralf@linux-mips.org>