Fix use-after-free in audio frag overlap

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Update README

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Tag version 1.5

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add Debian build instructions for 22.04

Set GLIB_VERSION_MIN_REQUIRED and GLIB_VERSION_MAX_ALLOWED

The minimum is fairly ancieht because we build some gtk-2.0 and EDS bits,
and perhaps we could do that *only* for those components as the rest of
the code is clean. But this is OK for now.

Add a maximum too, to help avoid using stuff which is too new to be usable
on ancient distributions we might still want to support. Like g_memdup2().

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Revert "use g_memdup2"

This reverts commit 51efa090a8d3153ce2012d04c77e5e93bee8c8a9.

The g_memdup2() function hasn't been around for that long, and we aren't
going to hit the case where it matters. Just tell glib not to whine.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Remove volatile from gst_meta_xcbimage_api_get_type()

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Fix prototype of do_conv_deliver_msg()

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Remove unused eparse label

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Fix join_v3 node parsing

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

use g_memdup2

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Fix GSourceFunc casts

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Remove unused eparse label

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Remove 'volatile' from CHIME_DEFINE_ENUM_TYPE

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Use v3 meetings API for finding meetings by PIN

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add v3 join_meeting support

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add meeting_authz support

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Implement mute-on-join

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Track mute_on_join_by for meetings

Don't do anything with it yet; just note it.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Change 'muted' to 'silent' in chime-call where that's what we mean

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Changed 'muted' to 'audio' flag in chime-meeting.c

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Changed 'muted' to 'audio' flag in prpl meeting handling

It's not really "muted"; it really is for no audio at all. And we want
to support mute on join so we'll care. Flip the polarity and call it
the 'audio' flag.

Leaving chime_connection_join_meeting_async() alone for now.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Rework batch fetching of messages to avoid GTimeVal

Slightly more than a trivial replacement of GTimeVal, keep the GDateTime
around in msg->fetch_until_dt instead of recreating it for each batch.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Eliminate GTimeVal from chimeseen.c

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Eliminate GTimeVal from rooms.c

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Eliminate GTimeVal from chime-room.c

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Eliminate GTimeVal from chat.c

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Kill parse_time() and replace it with iso8601_to_ms()

GTimeVal has been deprecated; we need to use GDateTime to parse ISO8601
strings. All we really want is milliseconds so just create a helper to
do that, and convert the one user of parse_time() to it for now. Other
callers of g_time_val_from_iso8601() will use it next...

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Remove deprecated g_type_class_add_private() for ChimeConnection

Sadly we then have to add an accessor function for it, which shows just
how non-private it actually is...

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Update login to use external web browser

The login process changed again and this time we really do need a browser.

Use purple_notify_uri() to spawn it, and now we get to specify our own
callback_uri for it to redirect to with the token. Use chime:// because
we already have the scripting set up to capture that.

This means we can fix the problem of multiple accounts with the chime:
scheme handler, by passing the devtoken of the account in question. We
can also pass the UI name so that we know which UI to re-enable it in.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Don't run autogen.sh from debian rules

Let's use 'make tmp-dist' to prepare tarballs.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Use make variables in debian/rules.maint

This makes it easier to override them when doing an out-of-tree build
after 'make tmp-dist'.

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Use POSIX tar format for dist

Tag version 1.4.1

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Add markdown.h to dist

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add desktop file to EXTRA_DIST

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add scripts to EXTRA_DIST

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Tag version 1.4

Signed-off-by: David Woodhouse <dwmw2@infradead.org>

Fix meeting ended signal handling

The arguments for the handler have *always* been wrong; it only ever
worked by fluke because the 'conn' must have ended up in the right
register for the third argument.

Add hackery for authentication via browser

Rename joinable.py to chime-joinable.py and install it

Merge branch 'deb-updates' of github.com:nmeyerhans/pidgin-chime

packaging: templatize debian/control

Perform substitution of Build-Depends and Depends per Debian/Ubuntu
release.  Build and binary dependencies for several releases are
provided; additional releases are simple to add.

In order to generate the .deb packaging metadata, a workflow such as
the following can be used (note that this assumes apt >= 2.0, which
introduced the 'satisfy' subcommand; on earlier releases, use a
combination of dpkg-checkbuilddeps and manual apt invocations or
pbuilder(8)):

$ make -f debian/rules.maint debian/changelog
$ sudo apt satisfy --no-install-recommends build-essential \
  "$(grep-dctrl -n -s Build-Depends . debian/control)"
$ dpkg-buildpackage -uc -us

This should generate proper Debian binary and source packages
targeting the current distribution provided the following conditions
are met:
1. The build is happening within the target distribution
2. The make, git, lsb-release, and dctrl-tools are installed

debian/control is generated with Build-Depends and binary Depends
substitute from the appropriate debian/deps/build-* and
debian/deps/bin-* files, respectively.

debian/changelog is generated based on git tag data with various
release and version fields substituded with the appropriate values for
the current distribution environment (e.g. Ubuntu focal or Debian
buster).

Signed-off-by: Noah Meyerhans <nmeyerha@amazon.com>

packaging: update external links in debian/control

Add missing Vcs-* headers and correct the Homepage location

Signed-off-by: Noah Meyerhans <nmeyerha@amazon.com>

Added a new 'video' indicator

Also added a 'ping' handled for video; but it seems harmless and without
any affect :)

Fix CHIME_PROPS_NEWOBJ

Where nick != name the properties weren't getting set. For some reason
I wasn't seeing the warnings that Nacho was.

Fix leak of machine_id

Merge branch 'wip/nacho/fixups' of github.com:awslabs/pidgin-chime

signin: do not use a weird opaque structure

We can save the data inside the object itself if required.

chime-connection: set the default value of the server property

This is in case that we want to use g_object_new instead of the
default constructor

Add attachment upload support for chats

Based on PR#36 by Guilherme Melo but teaching Pidgin that chats can
have files instead of manually adding a separate 'Send File' menu item:
https://bitbucket.org/pidgin/main/pull-requests/701

Cope with screen==NULL in screen_appsink_new_sample()

I saw a crash with this once, in a long sequence of sharing from A to B
and then B to A testing screen sharing startup/teardown.

It looksl like a race condition when we change to the no_appsink_callbacks
but the callback is already in progress, and the cbdata gets set to NULL.

Fix EPEL7 build.

Yeah, we still can't use C99 unless we explicity ask for it.

prpl/markdown.c: In function 'do_markdown':
prpl/markdown.c:48:2: error: 'for' loop initial declarations are only allowed in C99 mode
  for (char *p = *outbound; *p; p++) {
  ^

Add libmarkdown dep to specfile

Merge github.com:phillipberndt/pidgin-chime

Update display name for newly-created peers as they're discovered

I said instantiate *conversations* when they're opened...

Include <string.h> for dbus.c

Instantiate conversations as soon as they're opened; not on first send

Refine criteria for setting PURPLE_MESSAGE_DELAYED flag

Not for *all* messages in the last day, but only for the *last* message, if
it's less than a day old.

Fetch ancient messages in 2-week chunks

Because the server doesn't play them in chronological order and we have to
sort them, this takes a lot of memory if we do years worth of messages in
one hit. Instead, fetch a heuristically reasonable sized amount at a time.

Fix msg_newer() which was returning the wrong answer, while splitting
out msg_newer_than().

Less beeping for ancient messages.

Add chime_conversation_get_created_on()

Don't fetch all rooms at startup if we haven't been mentioned.

This takes a huge amount of time for new clients.

Don't specify organiser to generate_comp() at all.

It was using 'Calendar' for me, and resulting in a non-editable meeting.

Having carefully chosing the right calendar, we should be able to leave
it as NULL and let the default for the calendar be used.

Do not try to get the attachment if the node is NULL

This fixes lots of warnings when trying to get the attachments

Allow joinable meetings to be added

This paves the way for something that looks at the calendar and inserts
the meetings where the organiser forgot to invite meet@chime.aws, or
where distribution lists were invited instead of people, so Chime can't
pop up automatically.

Merge branch 'fix_Evolution_email_mismatch' of https://github.com/dlenski/pidgin-chime

Add End Meeting support

modify Evolution plugin to ensure that a calendar event is always associated with the email address of the source for that calendar

This addresses issue #44, wherein Evolution prevents the user from editing
the created meeting if the Chime username/address doesn't exactly match an
email address configured in Evolution.

Fix for attachment ignore problem

xcbimagesrc: disconnect xcb when clearing the context

xcbimagesrc: remove unused method

Prevent double-close of meetings when chat is closed.

xcb: gather all the screen data using xcb_screen_t

Fix 'Join with audio' on already-open meeting, ending audio closes chat

shm check

wip

Add gst-xcbimagesrc to specfile

Add xcbimagesrc for Nacho to port to XCB

It's still just a copy of ximagesrc from gst-plugins-good, passed through sed.

Fix potential memory leak in screen_send_packet()

Not that we use it in that mode, because it sucks as noted even with
the leak fixed.

No really, only send screen if there's a viewer.

Handle unexpected closure of screen websocket

I don't think this was a cause of dropouts because the reported behaviour
was a stall (no movement in the picture). And when the websocket drops,
the shared screen drops and other participants see that it's gone.

But we should handle it appropriately anyway. By visibly dropping and
reporting an error for now. If necessary we'll make it automatically
reconnect like the audio stream does.

Only send screen data when a viewer is actually present

That is, when we see the first KEY_REQUEST.

Merge branch 'master' of github.com:awslabs/pidgin-chime

Fix outbound screenshare dropouts

I think I've found the reason, along with the reason I could never
reproduce it on demand. The videorate element insists on producing
precisely one output frame for every third of the second... since the
epoch (when GStreamer started up). Not from the start of the call.

So if you've only been running Pidgin for a few seconds and you make
a test call, and it's the first GStreamer pipeline, it all works fine.

Hang up, wait an hour and then share another screen, and it'll send
10800 duplicate copies of the first frame before it ever sends anything
else.

Revert "Reduce outbound screenshare bandwidth"

This reverts commit c1d4c357b4ebda3c3d3faab9f2f21acdd385016b. It wasn't that.

chime-call-transport: use the right format type for size_t

Fix compiler warning in chime_connection_send_message_async()

 (1) Event suspicious_sizeof: Passing argument "&message" of type
     "gchar const **" and argument "8L /* sizeof (gchar const **) */"
     to function "g_checksum_update" is suspicious. In this case,
     "sizeof (gchar const **)" is equal to "sizeof (gchar const *)",
     but this is not a portable assumption.
 (2) Event remediation: Did you intend to use "sizeof (gchar const *)"
     instead of "sizeof (gchar const **)"?

chime-connection: remove useless check

The server variable is checked right before and set to SIGNIN_DEFAULT
if needed.

chat: set error to the right purple media

Fix clang static analysis warnings

Even though the code is ok better to deallocate the data
after it is removing it from the list, to avoid the warning
and to make the code clearer.

Reduce outbound screenshare bandwidth

I've been sharing my screen for an hour now, without any dropouts. Not
really sure if this is it; it's never really been reproducible on
demand when I've been trying to debug it anyway...

Fix use-after-free crash in message gathering

When cleanup_msgs() is called, there may be a pending fetch. We were
seeing crashes when fetch_msgs_cb() ended up using the freed msgs
structure. Let cleanup_msgs() do the freeing instead of the caller
(since we already have this nasty hack that the msgs struct needs
to be at the start of the chat/im structure. And then defer the
final free() until fetch_msgs_cb() is called, if necessary.

Stop i386 builds failing

We don't care about them much, but the PPA builds for i386 and the failures
are gratuitous and annoying.

Sort room list and implement 'Leave Room'

We can't leave rooms from the room list display because Pidgin doesn't
let us add action buttons there, but we could do our own version as a
search results box, triggered from the account menu.

Also haven't implemented 'delete room' yet.

Tag version 1.3

Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>

Add Do Not Disturb state

Also fix chime_availability_name() not to crash when it sees a new state
value that isn't in its list.

chat: do not crash if clicking IM without selecting a contact

Tag version 1.2

Signed-off-by: David Woodhouse <dwmw2@infradead.org>