Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix ETS leak on process death #6

Closed
wants to merge 1 commit into from

2 participants

@simonmacmullen

There's a tiny little leak here since ets:lookup/2 returns the entire row.

Well, one of our users got it up to 18GB... o_O.

@tonyg
Owner

Gawd how embarrassment. Thanks. Merged.

@tonyg tonyg closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 16, 2012
  1. @simonmacmullen
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  src/rfc4627_jsonrpc_registry.erl
View
2  src/rfc4627_jsonrpc_registry.erl
@@ -85,7 +85,7 @@ handle_info({'DOWN', _MonitorRef, process, DownPid, _Reason}, State) ->
case ets:lookup(?TABLE_NAME, {service_pid, DownPid}) of
[] ->
{noreply, State};
- [ServiceName] ->
+ [{_, ServiceName}] ->
ets:delete(?TABLE_NAME, {service_pid, DownPid}),
ets:delete(?TABLE_NAME, {service, ServiceName}),
{noreply, State}
Something went wrong with that request. Please try again.