Skip to content

Commit

Permalink
fix: security issues found by static scan (#11487)
Browse files Browse the repository at this point in the history
Summary:
- Possible memory leak

Pull Request resolved: #11487

Reviewed By: pedroerp

Differential Revision: D65763889

Pulled By: kevinwilfong

fbshipit-source-id: d50c47e80a2a4783e8a9193a801a7f733cb55f2b
  • Loading branch information
czentgr authored and facebook-github-bot committed Nov 12, 2024
1 parent c6843be commit 551fc2f
Showing 1 changed file with 19 additions and 0 deletions.
19 changes: 19 additions & 0 deletions velox/tpch/gen/dbgen/dbgen_gunk.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,18 @@
/*
* Copyright (c) Facebook, Inc. and its affiliates.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
/*
* Copyright owned by the Transaction Processing Performance Council.
*
Expand Down Expand Up @@ -57,6 +72,10 @@ static void cleanup_dist(distribution* target) {
}
free(target->list);
}
/* Allocated from permute_dist */
if (target->permute) {
free(target->permute);
}
}

void cleanup_dists(void) {
Expand Down

0 comments on commit 551fc2f

Please sign in to comment.