| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
strncpy(3) is not so safe function and can lead to mistakes. For
example, strncpy(dest, "Profanity", 10); is redundant and leads to
problems when someone changes the source string.
Different example is when 3rd argument equals to length of the
destination buffer. strncpy(3) doesn't terminate string with '\0' when
it truncates. Therefore, the destination string becomes corrupted.
Zeroize storage for 'nid', so the last byte remains '\0' in case of
truncate.
|
|
|
|
| |
strdup(3) requires string.h
|
|
|
|
| |
Regards https://github.com/profanity-im/profanity/issues/1367
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since https://github.com/profanity-im/profanity/commit/d92c576aa53505d712715b1acc6344af3262c84f
we rely on g_spawn_sync().
Which doesn't do variable/glob expansion.
For our use of call_external() in opening and URL or avatar this is
fine.
For getting the password we want to be able to use ~ for our files.
Let's use a shell here.
Fix https://github.com/profanity-im/profanity/issues/1364
|
|
|
|
| |
Check for pointer to be NULL before dereferencing it.
|
|
|
|
|
|
| |
"base" was not really base but the filename :-)
Fix https://github.com/profanity-im/profanity/issues/1362
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://github.com/profanity-im/profanity/commit/98c38dc6d6d29333c63f80327774f094610d8602
sets C99 as standard.
strdup() is not part of C99.
For now set `-D_POSIX_C_SOURCE=200809L` macro to have strdup() in C99.
Using `gnu99` instead would be another option.
We should take more care to use glib functions whenever possible.
Regards https://github.com/profanity-im/profanity/issues/1357
|
|
|
|
|
|
|
|
|
|
|
|
| |
Found this when looking to fix bug https://github.com/profanity-im/profanity/issues/1357
Not sure if it is related.
man 3 getenv sais:
```
As typically implemented, getenv() returns a pointer to a string within
the environment list. The caller must take care not to modify this
string, since that would change the environment of the process.
```
|
|
|
|
|
|
|
|
| |
Some systems don't provide recent libstrophe releases. When older
version of libstrophe is detected, don't build legacy auth support.
To simplify this patch, report about unsupported legacy auth and
keep commands option as is.
|
|
|
|
|
|
|
|
| |
New options:
/connect <account> [auth default|legacy]
/account <account> set auth default|legacy
Fixes #1236.
|
|
|
|
|
|
|
|
|
|
| |
Memleak was reinroduced in d92c576aa53505d712715b1acc6344af3262c84f
It was already fixed in ac5ce105ac08f022d88d7e73dc38f2706d4c44cf
But the rebase peetahs rebase ontop of master took the wrong changes.
I decided to pull anyways and fix since reviewing/giving feedback
without GH probably takes longer.
|
|
|
|
| |
Signed-off-by: Pierre Mazière <pierre.maziere@gmx.com>
|
|
|
|
| |
Signed-off-by: Pierre Mazière <pierre.maziere@gmx.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix https://github.com/profanity-im/profanity/issues/1280
p_contact_set_presence() is triggered in
(roster_process_pending_presence) to set the presence to online
after the roster is displayed.
This happened to me _every time_ with one certain account on my server.
But not even once with other accounts on my server.
I don't understand why that is the case, but AFAIK the code should be
like it is in this PR anyways.
roster_process_pending_presence() was added there in
973a05d15a9843f2e8f6dff598f2161367885994 to fix
https://github.com/profanity-im/profanity/issues/1050
|
| |
|
| |
|
|
|
|
| |
Fix https://github.com/profanity-im/profanity/issues/1348
|
|\ |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
NOTE: as per XEP-0060 §12.20, it is RECOMMENDED for the publisher to specify an
ItemID of "current" to ensure that the publication of a new item will overwrite
the existing item.
https://xmpp.org/extensions/xep-0384.html#devices
Issue: 1329
|
|\ \
| | |
| | | |
Incoming iq stanza - Reset the autoping timer
|
| |/
| |
| |
| |
| |
| |
| |
| | |
A autoping is to make sure that there is still a connection between server and
client. If the application receives incoming stanza, the connection is fine.
There is no need to wait for response, if there are other incomings.
Issue: #1333 and #1315
|
|\ \
| | |
| | | |
Bugfix: Status with PGP shows offline in titlebar
|
| |/
| |
| |
| |
| |
| |
| |
| | |
There is code to redraw the ui, because the user may enter a passphase for the
private key. There was also a ui_init, which shouldn't be called, because it
will set the status to the initial state, which is 'offline' and 'no tls'.
Issue: #1327
|
| |
| |
| |
| | |
Regards https://github.com/profanity-im/profanity/issues/1342
|
| |
| |
| |
| |
| |
| |
| |
| | |
Mistake introduced in 86f0e0ea7033a27a83a5c24091ae1fd5b579296d
The functions later on dont set an explitic position when writing their
strings.
Fix https://github.com/profanity-im/profanity/issues/1346
|
| | |
|
| | |
|
| |
| |
| |
| | |
Instead of connection_get_fulljid() and then creating a Jid from it.
|
| |
| |
| |
| | |
Regards https://github.com/profanity-im/profanity/issues/1115
|
| |
| |
| |
| | |
Regards https://github.com/profanity-im/profanity/issues/1115
|
| |
| |
| |
| |
| |
| | |
`/bookmarl ignore` lists the ignored bookmarks.
Regards https://github.com/profanity-im/profanity/issues/1115
|
| | |
|
| |
| |
| |
| | |
Introduced in 36f2569e53e67fda6023c5ec5f6a7714cbce1946
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
```
cat ~/.local/share/profanity/bookmark_ignore
[jubalh@iodoru.org]
profanity@rooms.dismail.de=true
[testuser@domain.org]
testr@rooms.domain.org=true
```
Regards https://github.com/profanity-im/profanity/issues/1115
|
|/
|
|
|
|
|
|
|
|
| |
```
~/.local/share/profanity % cat bookmark_ignore
[ignore]
profanity@rooms.dismail.de=
```
Regards https://github.com/profanity-im/profanity/issues/1115
|
|
|
|
| |
Fix https://github.com/profanity-im/profanity/issues/715
|
|
|
|
|
|
|
| |
So far we only had the logging in sv_ev_incoming_carbon() not
sv_ev_outgoing_carbon().
Regards https://github.com/profanity-im/profanity/issues/1342
|
|
|
|
|
| |
Fix 88c36745fe3f568f09fcd4078fc760a52e826eb7
Copy paste mistake.
|
|
|
|
| |
Implement https://github.com/profanity-im/profanity/issues/1326
|
|
|
|
|
|
|
|
| |
`/bookmark add|update` got `name` field.
By default localpart of JID is used (like before) but now we can set the
name ourselves.
Regards https://github.com/profanity-im/profanity/issues/697
|
|
|
|
|
|
| |
Have field in struct and display the name in `/bookmark list`.
Regards https://github.com/profanity-im/profanity/issues/697
|
|
|
|
|
|
|
| |
`statusbar.show.read` can be set to false in the config.
`/statusbar show|hide read`.
Implement https://github.com/profanity-im/profanity/issues/1285
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With d1d0ad8d1a8e28690aa8723566dd64c1ccdcf9d6 we set a timestamp to now
for all messages upon receival. Even if the original message didn't
contain any timestamp. So we can use the timestamp of retrieval for
logging and displaying and don't get them at each of those location
where they might differ.
This means that timestamp will never be NULL.
I don't see why we would want to check for the chat state only if
timestamp isn't there.
Probably because in XEP-0085 there is not timestamp defined.
So if it thats stanza it's not there and we can parse quicker, but there
is nothing forbidden it to be there.
Related to https://github.com/profanity-im/profanity/issues/1339
|
| |
|
|
|
|
|
|
| |
Display [SCROLLED] in titlebar if a chat/muc/private win is scrolled up.
Implement https://github.com/profanity-im/profanity/issues/1289
|
|
|
|
| |
So users are able to search the reference website easier.
|
|
|
|
|
| |
This is used to set the openers for various commands.
So far for /avatar and /urlopen.
|
| |
|
| |
|