ad_click table record insert fails if user agent string is longer than 255 chars

if ad.module line 240:
db_query("INSERT INTO {ad_clicks} (aid, uid, status, hostname, user_agent, adgroup, extra, hostid, url, timestamp) VALUES (%d, %d, %d, '%s', '%s', '%s', '%s', '%s', '%s', %d)", $aid, $user->uid, $status, ip_address(),$_SERVER['HTTP_USER_AGENT'], $group, $extra, $hostid, $url, time());

is changed like this:
db_query("INSERT INTO {ad_clicks} (aid, uid, status, hostname, user_agent, adgroup, extra, hostid, url, timestamp) VALUES (%d, %d, %d, '%s', '%s', '%s', '%s', '%s', '%s', %d)", $aid, $user->uid, $status, ip_address(), substr($_SERVER['HTTP_USER_AGENT'], 0, 255), $group, $extra, $hostid, $url, time());

It'd be avoidable. I suppose same goes for url also. Thanks!