fix CID 1596595:  Resource leaks, and CID 1596594:    (CHECKED_RETURN)

re-add Colin's attach pasted files commit, with the necessary fix

original commit was 66fccde959a1b4addee971412b35d4b51d8272b1, this includes the patch from Paul Wolneykien (bug 4260) which fixes the bug(s)

update copyright year

remove long unused TODO file

Remove unused includes

Completes previous patch :-)

Remove deprecated gtk_tree_view_set_rules_hint

This call has been deprecated since version 3.14 and removal doesn't
change current visual appearance. CSS support for this was removed 8
years ago, and it also seems strips in TreeViews are never going to
come back¹. The only option for now seems to use ListBox² after having
migrated to GTK 4…

¹ https://gitlab.gnome.org/GNOME/gtk/-/issues/581
² https://docs.gtk.org/gtk4/class.ListBox.html

Removes thirty “warning: ‘gtk_tree_view_set_rules_hint’ is deprecated”

Refactor definition and remove unused defines

Remove unused defines and normalise header file

Fix bug 4787 "Use correct function for memory transfer in crypt_cfb_buf"

There are really just 8 bytes of memory to transfer from one place to
another. No C strings involved.

warning: 'strncpy' output truncated before terminating nul copying 8
bytes from a string of the same length [-Wstringop-truncation]

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Fix bug 4786 "remove type confusion in getsockopt call in sock_connect_async_cb"

The function getsockopt expects a pointer to an area of memory whose
length has to specified in a variable of type socklen_t.

Adjust the type of optlen to be socklen_t as mandated by POSIX.

Adjust the type of optval pointer to what it really is, it will be
automatically converted to become a void * pointer.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Remove support for the obsolete AWN

The Avant Window Navigator released its latest version exactly fourteen
years ago. Package was removed from Debian three years later because of
bugs, and seems cairo-dock was the suggested alternative by Fedora.

This also removes the somewhat annoying warning:

“The name com.google.code.Awn was not provided by any .service files”

This partially reverts commit b2c06746b485e0f18834959f6aa2a635d78657e5

fix bug 4751, 'summaryview: remove translation hint from debug message'

fix bug 4773, 'remove obsolescent AC_C_CONST'

fix bug 4795, 'Please remove -no-cpp-precomp flag for Apple'

fix bug 4796, 'URL with wide character doesn't work'

Fix compiler warning.

fix bug 4788, '"Change primary passphrase" disabled status handling

fix bug 4789, 'Integrate single-setting dialog "Choose preset keyboard shortcuts"'
Moved keyboard binding preset selection from child dialog to main prefs_other window.
Update the combobox's sensitivity status to follow the 'Enable keyboard shortcuts' checkbox.
Select preset using combobox (as previously), click Apply or OK to activate the selected preset.
If 'Current' is the selected preset in combobox, Apply/OK won't change anything and keep current settings.

Add missing ketboard shortcuts to default preset in prefs_other (sync with tables in compose and main windows).

Add missing <control>E shortcut to compose window.

Normalized few <control><shift> to <shift><control>.

Better translation for NOOP error (thanks to Besnik_b).

Update to LiteHTML v0.9

Add OAUTH2 support for Microsoft 365 GCC High

Improve 4790 fix and normalise labels

fix bug 4790, 'widget spacing in "Changing primary passphrase" dialog' (to match what's was in GTK2, BTW).

Remove unused define

Support for metamail was dropped in 2.5.0 according to NEWS file.
Removed also two comments, very likely debugging remnants.

enable 'Remove references' when forwarding mail

fix bug 4754, 'text/enriched literal less-than sign sequence handled incorrectly' (by Viatrix)

fix bug 4747, 'matcher: simplify matcherprop_new' (mostly by Olaf Hering)

fix bug 4791, 'remove obsolete glib version check' (parts by Olaf Hering)

Fix bug 4749: release regex_t in summary_compile_simplify_regexp

Fix bug #4746: matcher: remove incorrect condition in matcherprop_free
In fact, Regex'es have been reenabled in Windows since 2015 (Andrej, 22a32c92603ba73754af64cd3b40de3d7c2f752a)
after they've been disabled in 2013 (Colin, 7f27738df23b2098214d956e6b41149c1a158404);
reenable everything of their handling in matcher.[ch].

fix several USE_AFTER_FREE coverity bugs

CIDs 1491105, 1491164, 1491139, 1491166, 1491168, 1491169, 1491178, 1491232, 1491242, 1492281

Fix bug #4782 use correct prototype for stop_archiving. Patches by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Fix bug 4781, 'use correct prototype for privacy_free_signature_data', patch by Olaf Hering

Fix bug #4780 use proper prototype for two archiver functions. Patch by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Remove remaining references to dirent

which has become unused now.

Use g_dir calls for directory processing

Also fix building with --enable-more-archive-debug while at it.

Use a configure flag for archive plugin extra debug

Remove inclusion of sys/socket.h from nntp-thread.c

Nothing in this file uses declarations or defines from this header.
It looks like copy&paste from imap-thread.c in commit
81fd89ef633cec2d54b3cff170127cd8d88f544d

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Include sys/socket.h unconditional in imap-thread.c

The code uses socketpair, which according to socketpair(3p) needs this
header. Move it into the existing conditional, because the calling
function imap_threaded_connect_cmd is also wrapped in the same
conditional.

Currently the header is indirectly included via netdb.h.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Remove useless code

Remove wrong initialization from max_message_size

A valid value can only ever be positive. It was introduced correctly
with an unsigned type, but initialized incorrectly. Later the type was
changed to signed, instead of removing the incorrect initialization.

Restore the proper type, and remove the incorrect initial value.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

oauth2: show body for every HTTP request

Consider also the refresh request, not just the initial request for
obtaining tokens.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Handle more systems which lack support for the scanf m modifier

The 'm' modifier for scanf is an extension to ISO C. It was mentioned in
POSIX 1003.1 2008 (Issue 7). It is difficult to detect at compile time
if a libc has implemented support for this modifier. Extend the existing
list of systems which lack support for it with NetBSD and Dragonfly BSD.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Use CFLAGS provided by nettle.pc

In case nettle requires any CFLAGS, use them as well.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Fix CID 1491211 'Use after free'

The function prefs_set_block_label frees its pfile argument on error
(via prefs_file_close_revert calls) and also returns -1, so use the
returned value to check instead of the freed pointer.

Update function documentation while we're at it.

Fix CID 1491137 'Out-of-bounds access (OVERRUN)'

Fix CIDs 1491064 and 1491074

Two instances of 'Use after free (USE_AFTER_FREE)'

Fix CIDs 1220325, 1491306 and 1491315

Three instances of "Explicit null dereferenced (FORWARD_NULL)"

Sync Spanish manual

Add a couple of shortcuts to manual

various clean-ups and optimisations by Olaf Hering

bug 4752, 'Adjust incorrect debug_printf call in pgp plugins'
bug 4757, 'remove AX_FUNC_MKDIR'
bug 4758, 'remove unused check for bind_textdomain_codeset'
bug 4759, 'remove unused function checks from AC_CHECK_FUNCS'
bug 4760, 'use correct type for move_bar_id'
bug 4768, 'Adjust logic while evaluating enable_avatars'
bug 4770, 'remove intl from list of include directories'

Fix bug #4765 oauth2: only store smtp auth if authorization method is OAUTH2 fix bug #4766 oauth2: preserve the expiry value of SMTP auth type is not OAUTH2 patches by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Fix bug #4762 oauth2: preserve an existing refresh token patch by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

fix bug 4750, 'remove regcomp wrapper and call regcomp directly'

Fail earlier when certificate sizes differ

There's no need to allocate memory and waste cycles exporting
certificates to fail afterwards because sizes differ.

Fix bug 4734, 'ssl_certificate: remove unhelpful warnings from certificate check'

bump copyright year

fix bug 4733, 'Line breaks lost in headers'

Fix bug #Bug 4730 add some more debug output. Patch from Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Revert "Fix bug #4730. Patch by Olaf Hering"

This reverts commit 91ebb5e00b27ab36b811a446f17b13cb78776395.

Fix bug #4730. Patch by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Fix Bug #4728. Patch provided by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Fix Bug #4725. Patch provided by Olaf Hering

Signed-off-by: Michael Rasmussen <mir@datanom.net>

Fix bug 4724: set proper availability status to sign/encrypt toolbar buttons (patch by Olaf Hering).

Improve English, thanks Paul for proofreading!

Make string easier to translate

Providing the whole sentence in one string will hopefully avoid bugs lke
https://www.claws-mail.org/bug/4722

rsvg_handle_get_intrinsic_size_in_pixels() is rsvg 2.52.

Remove Olaf (already known above in list as Olaf Hering).

Feed the contributors list.

Fix bug 4720 (resource leak), patch by olaf@aepfle.de.

Don't use deprecated calls with modern SVG library

Removes the following warnings:
• stock_pixmap.c:545:9: warning: ‘rsvg_handle_get_dimensions’ is
  deprecated: Use 'rsvg_handle_get_intrinsic_size_in_pixels' instead
• stock_pixmap.c:554:9: warning: ‘rsvg_handle_render_cairo’ is deprecated:
  Use 'rsvg_handle_render_document' instead
• stock_pixmap.c:675:17: warning: ‘rsvg_handle_get_dimensions’ is
  deprecated: Use 'rsvg_handle_get_intrinsic_size_in_pixels' instead

fix menu item text

fix bug 4668, 'Sometimes, at program start, message list takes all the vertical space' and other sizing issues

fix text of warning message

of course, we're copying a file not a message

continue if we can't create a backup of menurc

add a missed g_free()

fix paste errors introduced in 07245a4ad3a7431561c939c35b77df60cd2746de

fix paste errors introduced in 07245a4ad3a7431561c939c35b77df60cd2746de

remove a couple long unused file defs

fix paste errors introduced in last commit

revert menu label changes, give translators context instead

this avoids breaking custom user shortcuts on these menu items

make a backup of menurc on startup

Fix option names the preference is useful for

Document "nav_history_length" hidden preference

Add the mark all read/unread items back to the message list context menu.

Rename also “Color label” to “Color labels”

As suggested by Paul on translators ML.

Rename grouping menu option “Mark” to “Marks”

As requested on translators list. Menu paths also updated to match.

Distinguish messages and remove unnecessary goto

Fix building on non-X11 systems

Gtk Sockets and the GDK_IS_X11_DISPLAY() macro are only available when
GDK_WINDOWING_X11 is defined

if cmd is invalid, NULL it

this prevents an execution attempt of an invalid command, and also prevents spewing text content into the debug output, and also prevents it being necessary to hit the Cancel button twice on the 'Open with...' dialogue

for release 4.2.0

updated translations by David Medina, David Vachulka, Łukasz Wojniłowicz, Frederico Goncalves Guimaraes, Hugo Carvalho, Mikhail Kurinnoi, Slavko, Andreas Rönnquist, Numan Demirdöğen, and Mark Chang

add missing file

Update TLDs for release

fix bug 4361, 'Embedding external editor crashes Claws-Mail on Wayland' and a similar crash with the dillo plugin

Update French translation.

Update Spanish translation

More parentheses for yet another warning

summaryview.c:3986:66: warning: suggest parentheses around ‘&&’ within ‘||’

Add parentheses to fix warning

mimeview.c:1946:39: warning: suggest parentheses around ‘&&’ within ‘||’

don't build or distribute the gdata plugin