Cleanup scoped_ptr in src/common/dns_utils.cpp and header include:

* No need for entire new class (Fix #8732).
	* Fix stdint.h header include in contrib/epee/include/net/http_base.h.
This commit is contained in:
0xFFFC0000 2023-12-19 16:54:37 +03:30
parent ac02af9286
commit 430b102aef
2 changed files with 11 additions and 34 deletions

View File

@ -34,6 +34,7 @@
#include <string>
#include <utility>
#include <list>
#include <stdint.h>
#undef MONERO_DEFAULT_LOG_CATEGORY
#define MONERO_DEFAULT_LOG_CATEGORY "net.http"

View File

@ -28,6 +28,7 @@
#include "common/dns_utils.h"
// check local first (in the event of static or in-source compilation of libunbound)
#include "misc_language.h"
#include "unbound.h"
#include <deque>
@ -196,35 +197,6 @@ boost::optional<std::string> tlsa_to_string(const char* src, size_t len)
return std::string(src, len);
}
// custom smart pointer.
// TODO: see if std::auto_ptr and the like support custom destructors
template<typename type, void (*freefunc)(type*)>
class scoped_ptr
{
public:
scoped_ptr():
ptr(nullptr)
{
}
scoped_ptr(type *p):
ptr(p)
{
}
~scoped_ptr()
{
freefunc(ptr);
}
operator type *() { return ptr; }
type **operator &() { return &ptr; }
type *operator->() { return ptr; }
operator const type*() const { return &ptr; }
private:
type* ptr;
};
typedef class scoped_ptr<ub_result,ub_resolve_free> ub_result_ptr;
struct DNSResolverData
{
ub_ctx* m_ub_context;
@ -328,8 +300,12 @@ std::vector<std::string> DNSResolver::get_record(const std::string& url, int rec
dnssec_available = false;
dnssec_valid = false;
// destructor takes care of cleanup
ub_result_ptr result;
ub_result *result;
// Make sure we are cleaning after result.
epee::misc_utils::auto_scope_leave_caller scope_exit_handler =
epee::misc_utils::create_scope_leave_handler([&](){
ub_resolve_free(result);
});
MDEBUG("Performing DNSSEC " << get_record_name(record_type) << " record query for " << url);