Age | Commit message (Collapse) | Author |
|
Remove a couple unused functions
|
|
Prevent a OOB read when parsing IRCNet ! channels
See merge request irssi/irssi!22
|
|
Don't proceed with cmd_msg if there was an error splitting msg
See merge request irssi/irssi!21
|
|
fix dcc
See merge request irssi/irssi!20
|
|
This avoids undefined behavior in functions that call these split
functions and expect an array back instead of just a NULL pointer.
|
|
Fix use after free when channel is destroyed inbetween being synced
See merge request irssi/irssi!19
|
|
This reverts commit bd83852d646de28f2e0fe01efe7c9236aa4074d4.
|
|
fix out of bounds read in compress_colors
See merge request irssi/irssi!18
|
|
There may be cases (such as if target or server->nick is very long)
where the split_message function returns NULL, indicating an error. To
avoid a potential segfault, we now check to see if splitmsgs is NULL.
|
|
|
|
Make sure the string has enough data.
Fixes #16
|
|
Add alternate_nick as a network-specific property
|
|
|
|
|
|
This reverts commit cb5f3cba1f2b5dbad67bcc107f3fe4a1875cc52d.
|
|
Set host to an empty string on error
|
|
|
|
|
|
|
|
|
|
This is needed for servers like ssl.efnet.org, which, per default
config, listen on 9999.
|
|
This also removes a wordy comment
|
|
|
|
|
|
|
|
While investigating #317, I noticed that it was possible we would access
an uninitialized buffer due to failing to check the return value of
net_ip2host(). This is done in several places. To make such uses safe,
set the host buffer to an empty string on error. It is possible callers
could be improved by handling the error in each spot, but this gives us
some safety.
|
|
It is possible for rec to be NULL in query_remove_all, resulting in a
segfault. So return without doing anything if rec is NULL.
|
|
the chanquery needs to be removed in any case if a channel rec is
destroyed, regardless of any state
Fixes GL#13
|
|
Reported by Hanno Böck.
Fixes GL#12
|
|
This requires FreeBSD fix (https://reviews.freebsd.org/D12622)
to work properly.
|
|
add new function to set the position in bytes
|
|
|
|
fixes #752
|
|
Get rid of the zombies in Capsicum capability mode.
|
|
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
Let's implement X509_STORE_up_ref on our own.
|
|
OpenSSL doesn't increment the reference count when the store is assigned
to a SSL_CTX.
|
|
Add Capsicum support
|
|
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
|
|
Revert "Merge pull request #452 from LemonBoy/terminfo-cup"
|
|
This restores the previous behaviour.
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
Fix indentation; no functional changes.
|
|
(Take two.)
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
Fixes #733. The fix outlined in #452 had adverse effects for the
following reason. The code removed the restoration path that would go on
the code path from kill SIGTSTP. The problem is this: When Irssi is not
running in a controlling parent (like a shell), the TSTP will in fact be
ignored. In that case, there is no process sending a CONT either and
thus the screen state never gets restored. Luckily, the patch in #457 is
sufficient to prevent the problem in #450 (which lead to the development
of #452). To that end, we do end up with potentially calling
terminfo_cont twice but that is better than not calling it at all.
This reverts commit b1ffd5f6472584aa3966746da9728c5afefcc4ce, reversing
changes made to 9cb0419435d1ad331c1f55361a003d9682fae9a8.
|
|
|
|
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|
|
Signed-off-by: Edward Tomasz Napierala <trasz@FreeBSD.org>
|